get_file_info(): Don't dereference out of scope char array.

A pointer to `char buf[...]' is taken, the array being declared in a
block scope inside the function.  The pointer is passed to strdup(3) as
the function returned, but the array is by then out of scope.  Move the
array to the function's scope.  Fixes 0c50c669.

diff --git a/sbr/mime_type.c b/sbr/mime_type.c
index f1b96fe591c001ca6ac3885a11701ea4dec3c03a..25928500cdd8751a2b034cc2718fca7010d4f178 100644 (file)
--- a/sbr/mime_type.c
+++ b/sbr/mime_type.c
@@ -117,6 +117,7 @@ static char *
 get_file_info(const char *proc, const char *file_name) {
     char *cmd, *cp;
     char *quotec = "'";
+    char buf[max(BUFSIZ, 2048)];
 
     if ((cp = strchr(file_name, '\''))) {
         /* file_name contains a single quote. */
@@ -132,7 +133,6 @@ get_file_info(const char *proc, const char *file_name) {
         FILE *fp;
 
         if ((fp = popen(cmd, "r")) != NULL) {
-            char buf[max(BUFSIZ, 2048)];
 
             buf[0] = '\0';
             if (fgets(buf, sizeof buf, fp)) {