|
From: <sv...@va...> - 2011-05-04 09:07:46
|
Author: sewardj
Date: 2011-05-04 10:07:38 +0100 (Wed, 04 May 2011)
New Revision: 11724
Log:
setup_client_stack: use have_exename to consistently guard uses
of VG_(args_the_exename), thereby avoiding a potential segfault.
Spotted by IBM's BEAM checker.
Modified:
trunk/coregrind/m_initimg/initimg-linux.c
Modified: trunk/coregrind/m_initimg/initimg-linux.c
===================================================================
--- trunk/coregrind/m_initimg/initimg-linux.c 2011-05-04 09:06:17 UTC (rev 11723)
+++ trunk/coregrind/m_initimg/initimg-linux.c 2011-05-04 09:07:38 UTC (rev 11724)
@@ -456,7 +456,7 @@
stringsize += VG_(strlen)(cauxv->u.a_ptr) + 1;
else if (cauxv->a_type == AT_RANDOM)
stringsize += 16;
- else if (cauxv->a_type == AT_EXECFN)
+ else if (cauxv->a_type == AT_EXECFN && have_exename)
stringsize += VG_(strlen)(VG_(args_the_exename)) + 1;
auxsize += sizeof(*cauxv);
}
|