[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
- [SCM] Debian GNU Hurd packaging branch, upstream, updated. upstream/hurd/20130707-5-gde8b182, Samuel Thibault, 2013/07/28
- [SCM] Debian GNU Hurd packaging branch, upstream, updated. upstream/hurd/20130707-5-gde8b182,
Samuel Thibault <=
- [SCM] Debian GNU Hurd packaging branch, upstream, updated. upstream/hurd/20130707-5-gde8b182, Samuel Thibault, 2013/07/28
- [SCM] Debian GNU Hurd packaging branch, upstream, updated. upstream/hurd/20130707-5-gde8b182, Samuel Thibault, 2013/07/28
- [SCM] Debian GNU Hurd packaging branch, upstream, updated. upstream/hurd/20130707-5-gde8b182, Samuel Thibault, 2013/07/28