CID:1214192 Dereference after null check, don't crash when no file name passed on file open

2014-05-15 13:37:26 +00:00 · 2014-05-15 13:37:26 +00:00 · 92c6f3abc5
parent 179e07c649
commit 92c6f3abc5
1 changed files with 5 additions and 0 deletions
--- a/src/mod/applications/mod_httapi/mod_httapi.c
+++ b/src/mod/applications/mod_httapi/mod_httapi.c
@ -2828,6 +2828,11 @@ static switch_status_t file_open(switch_file_handle_t *handle, const char *path,
 			}	
 		}

+		if (!context->write.file_name) {
+			switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "No file name specified.\n");
+			return SWITCH_STATUS_GENERR;
+		}
+
 		if ((ext = strrchr(context->write.file_name, '.'))) {
 			ext++;
 		} else {