commit-hurd
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[SCM] Debian GNU Hurd packaging branch, upstream, updated. upstream/hurd


From: Samuel Thibault
Subject: [SCM] Debian GNU Hurd packaging branch, upstream, updated. upstream/hurd/20130707-5-gde8b182
Date: Sat, 27 Jul 2013 22:25:33 +0000

The following commit has been merged in the upstream branch:
commit 5534751bfc13761fd87c667cd3f3719575adac2a
Author: Justus Winter <address@hidden>
Date:   Thu Jul 11 18:09:05 2013 +0200

    libnetfs: fix consistency check
    
    passive is not a zero terminated string but a char * combined with a
    length. If passivelen == 0, passive may very well be not NULL, and
    dereferencing that pointer might not be safe. At the very least the
    consistency check is wrong. Fix that by checking passivelen instead of
    passive.
    
    * libnetfs/file-set-trans.c (netfs_S_file_set_translator): Fix sanity check.

diff --git a/libnetfs/file-set-translator.c b/libnetfs/file-set-translator.c
index 4c29d95..b107ccd 100644
--- a/libnetfs/file-set-translator.c
+++ b/libnetfs/file-set-translator.c
@@ -39,7 +39,7 @@ netfs_S_file_set_translator (struct protid *user,
   if (!(passive_flags & FS_TRANS_SET) && !(active_flags & FS_TRANS_SET))
     return 0;
 
-  if (passive && passive[passivelen - 1])
+  if (passivelen && passive[passivelen - 1])
     return EINVAL;
 
   np = user->po->np;

-- 
Debian GNU Hurd packaging



reply via email to

[Prev in Thread] Current Thread [Next in Thread]