summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Schneider <asn@samba.org>2013-02-25 15:41:23 +0100
committerDavid Disseldorp <ddiss@suse.de>2013-03-05 23:30:05 +0100
commitceb1d8b84a6bd7084447e4cc75bd6821d6fdbe34 (patch)
tree7bcc4e4658977cb02624a6d23ba2cd195a488286
parentcab969d5ec67b1b3afbd4be2ff3a0d3cec45fd45 (diff)
downloadsamba-ceb1d8b84a6bd7084447e4cc75bd6821d6fdbe34.tar.gz
samba-ceb1d8b84a6bd7084447e4cc75bd6821d6fdbe34.tar.bz2
samba-ceb1d8b84a6bd7084447e4cc75bd6821d6fdbe34.zip
gensec: recv_handler can't be NULL at that point.
We probably want to segfault here if it is NULL. Reviewed-by: David Disseldorp <ddiss@samba.org>
-rw-r--r--source4/auth/gensec/socket.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source4/auth/gensec/socket.c b/source4/auth/gensec/socket.c
index 99b4108a39..c89e080232 100644
--- a/source4/auth/gensec/socket.c
+++ b/source4/auth/gensec/socket.c
@@ -111,7 +111,7 @@ static void gensec_socket_trigger_read(struct tevent_context *ev,
/* It may well be that, having run the recv handler, we still
* have even more data waiting for us!
*/
- if (gensec_socket->read_buffer.length && gensec_socket->recv_handler) {
+ if (gensec_socket->read_buffer.length > 0) {
/* Schedule this funcion to run again */
tevent_add_timer(gensec_socket->ev, gensec_socket, timeval_zero(),
gensec_socket_trigger_read, gensec_socket);