]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
vol: initialize FdHandle_t stack objects
authorJeffrey Altman <jaltman@your-file-system.com>
Sat, 15 Jan 2011 16:39:39 +0000 (11:39 -0500)
committerDerrick Brashear <shadow@dementia.org>
Thu, 3 Feb 2011 20:02:52 +0000 (12:02 -0800)
In namei_create() a temporary FdHandle_t is allocated on
the stack but was never initialized.  Depending on what
prior data was on the stack the FdHandle_t could result
in subsequent operations failing in unpredictable ways.

Reviewed-on: http://gerrit.openafs.org/3667
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
(cherry picked from commit 05d5f012e0be9be1c2c82871a75ac76d437b9688)

Change-Id: I44913419e09b36a3581320cd169bbc4774cd0208
Reviewed-on: http://gerrit.openafs.org/3829
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
src/vol/namei_ops.c

index 56d32278a902d595185aef9760546e51e7bb292c..452213af576f9b5ed0f53b14463da5502681f17d 100644 (file)
@@ -751,6 +751,7 @@ namei_icreate(IHandle_t * lh, char *part, afs_uint32 p1, afs_uint32 p2, afs_uint
     b32_string_t str1;
 
     memset((void *)&tmp, 0, sizeof(IHandle_t));
+    memset(&tfd, 0, sizeof(FdHandle_t));
 
     tmp.ih_dev = nt_DriveToDev(part);
     if (tmp.ih_dev == -1) {
@@ -876,6 +877,7 @@ namei_icreate(IHandle_t * lh, char *part, afs_uint32 p1, afs_uint32 p2, afs_uint
     int ogm_parm;
 
     memset((void *)&tmp, 0, sizeof(IHandle_t));
+    memset(&tfd, 0, sizeof(FdHandle_t));
 
     tmp.ih_dev = volutil_GetPartitionID(part);
     if (tmp.ih_dev == -1) {