git://git.openafs.org / openafs.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 2608ce6) | patch
ubik: Avoid use of freed string
authorMarc Dionne <marc.dionne@your-file-system.com>
Thu, 9 May 2013 16:30:39 +0000 (12:30 -0400)
committerJeffrey Altman <jaltman@your-file-system.com>
Fri, 10 May 2013 19:00:44 +0000 (12:00 -0700)
commit641b2fd8d6a270a2fc202450ab8dce044afc48a2
treeee88b5c6221f0e90148d45798168f44856750499tree | snapshot
parent2608ce67f1ba5ed1ed63525fad90db9d67547ec6commit | diff
ubik: Avoid use of freed string

If cellName was just set to dir->cellName, afsconf_Close() will
free the string before fprintf uses it.  Just change the order
so we don't access freed memory and print garbage.

Change-Id: I66792dc50addf2d8242f71923db09fcc178cc450
Reviewed-on: http://gerrit.openafs.org/9886
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Simon Wilkinson <simonxwilkinson@gmail.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
src/ubik/uinit.c diff | blob | history
OpenAFS Master Repository