[SCM] Samba Shared Repository - branch master updated
Simo Sorce
idra at samba.org
Mon Feb 14 09:48:01 MST 2011
The branch, master has been updated
via d6a41c2 Return NULL if tevent_add_fd() is passed a negative fd
from b423d83 s4:ldapcmp: cope with range retrivals of multivalued attributes
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master
- Log -----------------------------------------------------------------
commit d6a41c2723cda59b3cd6ae9a1e77b62f25067663
Author: Stephen Gallagher <sgallagh at redhat.com>
Date: Mon Feb 14 10:29:49 2011 -0500
Return NULL if tevent_add_fd() is passed a negative fd
Autobuild-User: Simo Sorce <idra at samba.org>
Autobuild-Date: Mon Feb 14 17:47:03 CET 2011 on sn-devel-104
-----------------------------------------------------------------------
Summary of changes:
lib/tevent/tevent_fd.c | 6 ++++++
1 files changed, 6 insertions(+), 0 deletions(-)
Changeset truncated at 500 lines:
diff --git a/lib/tevent/tevent_fd.c b/lib/tevent/tevent_fd.c
index c58e8e1..455961b 100644
--- a/lib/tevent/tevent_fd.c
+++ b/lib/tevent/tevent_fd.c
@@ -51,6 +51,12 @@ struct tevent_fd *tevent_common_add_fd(struct tevent_context *ev, TALLOC_CTX *me
{
struct tevent_fd *fde;
+ /* tevent will crash later on select() if we save
+ * a negative file descriptor. Better to fail here
+ * so that consumers will be able to debug it
+ */
+ if (fd < 0) return NULL;
+
fde = talloc(mem_ctx?mem_ctx:ev, struct tevent_fd);
if (!fde) return NULL;
--
Samba Shared Repository
More information about the samba-cvs
mailing list