commit a01347240c781d1c9cd4c4ed342796f7910597b9
author Paul Stewart <pstew@google.com> Sat Mar 19 05:03:05 2016 +0000
committer android-build-merger <android-build-merger@google.com> Sat Mar 19 05:03:05 2016 +0000
tree 6c356a1383abbc33fb86084b67aed6efe069fe96
parent 6d6d685fd8e7ace3d5d5f79aee86f192759a030c
parent e380dbeb0242a93e3fd6014fc53ea97f6a09daab

Guard against return value already being null am: 9c42c0b
am: e380dbe

* commit 'e380dbeb0242a93e3fd6014fc53ea97f6a09daab':
  Guard against return value already being null

wpa_supplicant/config.c

diff --git a/wpa_supplicant/config.c b/wpa_supplicant/config.c
index 6cd9614..f782b0d 100644
--- a/wpa_supplicant/config.c
+++ b/wpa_supplicant/config.c
@@ -2369,8 +2369,10 @@
 		const struct parse_data *field = &ssid_fields[i];
 		if (os_strcmp(var, field->name) == 0) {
 			char *ret = field->writer(field, ssid);
-			if (os_strchr(ret, '\r') != NULL || os_strchr(ret, '\n') != NULL) {
-				wpa_printf(MSG_ERROR, "Found newline in value for %s; "
+			if (ret != NULL && (os_strchr(ret, '\r') != NULL ||
+				os_strchr(ret, '\n') != NULL)) {
+				wpa_printf(MSG_ERROR,
+					"Found newline in value for %s; "
 					"not returning it", var);
 				os_free(ret);
 				ret = NULL;