Changes in / [967:bbc08d58325a:964:992ef1167344] in freeDiameter

File:

• libfdcore/fdd.y (modified) (1 diff)

libfdcore/fdd.y

@@ -323 +323 @@
                                                 { yyerror (&yylloc, conf, "Not enough memory"); YYERROR; } );
                                         sprintf(fname, DEFAULT_EXTENSIONS_PATH "/%s", bkp);
+                                        free(bkp);
                                         fd = fopen(fname, "r");
-                                        if (fd == NULL) {
-                                                free(fname);
-                                                fname = bkp;
-                                        } else {
-                                                free(bkp);
-                                        }
-                                }
-                                if (fd != NULL) {
+                                }
+                                if (fd == NULL) {
+                                        int ret = errno;
+                                        TRACE_DEBUG_ERROR("WARNING: Unable to open extension file %s for reading: %s\nLD_LIBRARY_PATH will be used.\n", fname, strerror(ret));
+                                } else {
                                         fclose(fd);
-                                } /* otherwise, LD_LIBRARY_PATH will be tested by dl_open. 
-                                This should not give any security issue, otherwise we can add an "else fail" here. */
+                                }
                                 
                                 /* Try and open the configuration file (optional) */