VG_(env_unsetenv), VG_(env_clone): add assertions so as to cause

assertions instead of segfaults.  Potential segfaults were detected by
IBM's BEAM checker.

git-svn-id: svn://svn.valgrind.org/valgrind/trunk@11721

diff --git a/coregrind/m_libcproc.c b/coregrind/m_libcproc.c
index e01f6129df5151275803af165c75e6758ca7faf3..dbedac2edef72a8931a04827b5fb84d6e7bd25ff 100644 (file)
--- a/coregrind/m_libcproc.c
+++ b/coregrind/m_libcproc.c
@@ -91,8 +91,10 @@ Char *VG_(getenv)(Char *varname)
 
 void  VG_(env_unsetenv) ( Char **env, const Char *varname )
 {
-   Char **from;
-   Char **to = NULL;
+   Char **from, **to;
+   vg_assert(env);
+   vg_assert(varname);
+   to = NULL;
    Int len = VG_(strlen)(varname);
 
    for (from = to = env; from && *from; from++) {
@@ -302,6 +304,7 @@ Char **VG_(env_clone) ( Char **oldenv )
    Char **newenv;
    Int  envlen;
 
+   vg_assert(oldenv);
    for (oldenvp = oldenv; oldenvp && *oldenvp; oldenvp++);
 
    envlen = oldenvp - oldenv + 1;