json-file: improve error handling

If the functions getting uri, etc fail. Return "unknown" instead of
NULL pointer. This improves consistency.

src/output-json-file.c

@@ -86,9 +86,10 @@ static json_t *LogFileMetaGetUri(const Packet *p, const File *ff) {
                 if (s != NULL) {
                     js = json_string(s);
                     SCFree(s);
+                    if (js != NULL)
+                        return js;
                 }
             }
-            return js;
         }
     }
 
@@ -105,10 +106,11 @@ static json_t *LogFileMetaGetHost(const Packet *p, const File *ff) {
             if (s != NULL) {
                 js = json_string(s);
                 SCFree(s);
-            }
+                if (js != NULL)
                     return js;
             }
         }
+    }
 
     return json_string("<unknown>");
 }
@@ -127,11 +129,12 @@ static json_t *LogFileMetaGetReferer(const Packet *p, const File *ff) {
                 if (s != NULL) {
                     js = json_string(s);
                     SCFree(s);
-                }
+                    if (js != NULL)
                         return js;
                 }
             }
         }
+    }
 
     return json_string("<unknown>");
 }
@@ -150,11 +153,12 @@ static json_t *LogFileMetaGetUserAgent(const Packet *p, const File *ff) {
                 if (s != NULL) {
                     js = json_string(s);
                     SCFree(s);
-                }
+                    if (js != NULL)
                         return js;
                 }
             }
         }
+    }
 
     return json_string("<unknown>");
 }