OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-3583-g8260d86
Gerrit Code Review
gerrit@openafs.org
Thu, 7 Mar 2013 07:58:16 -0800 (PST)
The following commit has been merged in the master branch:
commit 8260d86dda766a21e9f457994e7a3b259ba3a31b
Author: Simon Wilkinson <sxw@your-file-system.com>
Date: Sat Mar 2 11:02:09 2013 +0000
libadmin: Don't free garbage pointer
If we jump to the error handler early on in pts_GroupOwnerChange,
idlist may not have been used, and so we will end up trying to
free stack garbage.
Initialise the structure to 0 at the start of the function, so it
is always safe to enter the error handler.
Caught by coverity (#985962)
Change-Id: If70102e3da07135a9ec695f13caebe6298eff8ca
Reviewed-on: http://gerrit.openafs.org/9398
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
src/libadmin/pts/afs_ptsAdmin.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
--
OpenAFS Master Repository