Kill import if db name is invalid

morgando · morgando · commit b899f4c6ad2a · 2025-06-13T10:17:38.000-04:00
Signed-off-by: mdouglas47 &lt;mdouglas47@bloomberg.net&gt;
diff --git a/util/import_util.c b/util/import_util.c
@@ -18,7 +18,7 @@ enum bulk_import_validation_rc validate_bulk_import_inputs(const char * const de
     if (!str_is_alphanumeric(bulk_import_src_dbname, NON_ALPHANUM_CHARS_ALLOWED_IN_DBNAME)) {
         logmsg(LOGMSG_WARN, "%s: Bulk import source db name '%s' has illegal characters\n",
                 __func__, bulk_import_src_dbname);
-        rc = BULK_IMPORT_VALIDATION_WARN;
+        rc = BULK_IMPORT_VALIDATION_FATAL;
     }
     if (bulk_import_src_tablename && !str_is_alphanumeric(bulk_import_src_tablename, NON_ALPHANUM_CHARS_ALLOWED_IN_TABLENAME)) {
         logmsg(LOGMSG_WARN, "%s: Bulk import source table name '%s' has illegal characters\n",

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ enum bulk_import_validation_rc validate_bulk_import_inputs(const char * const de`
`18`	`18`	`if (!str_is_alphanumeric(bulk_import_src_dbname, NON_ALPHANUM_CHARS_ALLOWED_IN_DBNAME)) {`
`19`	`19`	`logmsg(LOGMSG_WARN, "%s: Bulk import source db name '%s' has illegal characters\n",`
`20`	`20`	`__func__, bulk_import_src_dbname);`
`21`		`- rc = BULK_IMPORT_VALIDATION_WARN;`
	`21`	`+ rc = BULK_IMPORT_VALIDATION_FATAL;`
`22`	`22`	`}`
`23`	`23`	`if (bulk_import_src_tablename && !str_is_alphanumeric(bulk_import_src_tablename, NON_ALPHANUM_CHARS_ALLOWED_IN_TABLENAME)) {`
`24`	`24`	`logmsg(LOGMSG_WARN, "%s: Bulk import source table name '%s' has illegal characters\n",`