Fix issue trusteddomainproject#121: Don't use implicit assignment tes…

…t in an "if".
devgs · Mar 18, 2021 · 31bb5f7 · 31bb5f7
1 parent 9fdbdf9
commit 31bb5f7
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/opendmarc/opendmarc-arcares.c b/opendmarc/opendmarc-arcares.c
@@ -192,7 +192,7 @@ opendmarc_arcares_parse (u_char *hdr, struct arcares *aar)
 		  case AAR_TAG_INSTANCE:
 			aar->instance = atoi(tag_value);
 			/* next value will be unlabeled authserv_id */
-			if (token = strsep((char **) &tmp_ptr, ";"))
+			if ((token = strsep((char **) &tmp_ptr, ";")) != NULL)
 			{
 				leading_space_len = strspn(token, " \n\t");
 				tag_value = opendmarc_arcares_strip_whitespace(token);