mid_registrar: Fix a NULL pointer dereferencing bug

Properly parse message headers before working on them Fixes #1088
OpenSIPS · Apr 3, 2017 · ed5b390 · ed5b390
1 parent 16cf1b5
commit ed5b390
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/modules/mid_registrar/save.c b/modules/mid_registrar/save.c
@@ -2213,6 +2213,8 @@ int mid_reg_save(struct sip_msg *msg, char *dom, char *flags_gp,
 		parse_save_flags(&flags_str, &sctx);
 	}
 
+	parse_reg_headers(msg);
+
 	if (!to_uri_gp) {
 		to_uri = get_to(msg)->uri;
 	} else if (fixup_get_svalue(msg, (gparam_p)to_uri_gp, &to_uri)) {
@@ -2233,8 +2235,6 @@ int mid_reg_save(struct sip_msg *msg, char *dom, char *flags_gp,
 		return -1;
 	}
 
-	parse_reg_headers(msg);
-
 	if (check_contacts(msg, &st) > 0) {
 		goto out_error;
 	}