Fix pointer accessing local variable out of scope

The C11 standard states that the value of a pointer is undefined
outside of its lifetime. Since we were initilizing the argv pointer
in two conditional blocks, compilers that implement this standard were
cleaning up the value and this caused to invalid dereferences of the
pointer.

The change introduces two local variables with the same lifetime of the
argv pointer and assigns these in the conditional blocks instead.

Signed-off-by: Evan Lezar <[email protected]>

Author: elezar

src/nvc_ldcache.c

@@ -472,6 +472,8 @@ nvc_ldcache_update(struct nvc_context *ctx, const struct nvc_container *cnt)
         if (validate_args(ctx, cnt != NULL) < 0)
                 return (-1);
 
+        char *argv_default[] = {cnt->cfg.ldconfig, "-f", "/etc/ld.so.conf", "-C", "/etc/ld.so.cache", cnt->cfg.libs_dir, cnt->cfg.libs32_dir, NULL};
+        char *argv_with_compat_dir[] = {cnt->cfg.ldconfig, "-f", "/etc/ld.so.conf", "-C", "/etc/ld.so.cache", cnt->cuda_compat_dir, cnt->cfg.libs_dir, cnt->cfg.libs32_dir, NULL};
         if ((cnt->flags & OPT_CUDA_COMPAT_MODE_LDCONFIG) && (cnt->cuda_compat_dir != NULL)) {
                 /*
                  * We include the cuda_compat_dir directory on the ldconfig
@@ -481,9 +483,9 @@ nvc_ldcache_update(struct nvc_context *ctx, const struct nvc_container *cnt)
                  * libs32_dir).
                  * */
                 log_info("prefering CUDA Forward Compatibility dir when running ldconfig");
-                argv = (char * []){cnt->cfg.ldconfig, "-f", "/etc/ld.so.conf", "-C", "/etc/ld.so.cache", cnt->cuda_compat_dir, cnt->cfg.libs_dir, cnt->cfg.libs32_dir, NULL};
+                argv = argv_with_compat_dir;
         } else {
-                argv = (char * []){cnt->cfg.ldconfig, "-f", "/etc/ld.so.conf", "-C", "/etc/ld.so.cache", cnt->cfg.libs_dir, cnt->cfg.libs32_dir, NULL};
+                argv = argv_default;
         }
 
         if (*argv[0] == '@') {