[svn-commits] kharwell: branch 13 r431693 - /branches/13/res/ari/ari_websockets.c

SVN commits to the Digium repositories svn-commits at lists.digium.com
Wed Feb 11 11:36:41 CST 2015


Author: kharwell
Date: Wed Feb 11 11:36:38 2015
New Revision: 431693

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=431693
Log:
ari_websockets: removed extra check on websocket session read

When merging the websocket timeout issue (ASTERISK-24701) an extra, almost
duplicate, check was left in the code that should not have been. This removes
it.

ASTERISK-24701 #close
Reported by: Matt Jordan
Review: https://reviewboard.asterisk.org/r/4412/

Modified:
    branches/13/res/ari/ari_websockets.c

Modified: branches/13/res/ari/ari_websockets.c
URL: http://svnview.digium.com/svn/asterisk/branches/13/res/ari/ari_websockets.c?view=diff&rev=431693&r1=431692&r2=431693
==============================================================================
--- branches/13/res/ari/ari_websockets.c (original)
+++ branches/13/res/ari/ari_websockets.c Wed Feb 11 11:36:38 2015
@@ -104,12 +104,6 @@
 		return NULL;
 	}
 
-
-	if (ast_websocket_fd(session->ws_session) <= 0) {
-		return NULL;
-	}
-
-
 	while (!message) {
 		int res;
 		char *payload;




More information about the svn-commits mailing list