[Asterisk-code-review] Fix IO conversion bug (asterisk[master])

Badalian Vyacheslav asteriskteam at digium.com
Thu Dec 8 12:36:21 CST 2016


Badalian Vyacheslav has uploaded a new change for review. ( https://gerrit.asterisk.org/4594 )

Change subject: Fix IO conversion bug
......................................................................

Fix IO conversion bug

Expression 'rlen < 0' is always false.
Unsigned type value is never < 0.

Change-Id: Id9f393ff25b009a6c4a6e40b95f561a9369e4585
---
M res/res_http_websocket.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/94/4594/1

diff --git a/res/res_http_websocket.c b/res/res_http_websocket.c
index 106ba48..8413823 100644
--- a/res/res_http_websocket.c
+++ b/res/res_http_websocket.c
@@ -490,7 +490,7 @@
  */
 static inline int ws_safe_read(struct ast_websocket *session, char *buf, int len, enum ast_websocket_opcode *opcode)
 {
-	size_t rlen;
+	ssize_t rlen;
 	int xlen = len;
 	char *rbuf = buf;
 	int sanity = 10;

-- 
To view, visit https://gerrit.asterisk.org/4594
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id9f393ff25b009a6c4a6e40b95f561a9369e4585
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Badalian Vyacheslav <v.badalyan at open-bs.ru>



More information about the asterisk-code-review mailing list