diff options
author | Jouni Malinen <j@w1.fi> | 2014-12-07 10:15:34 (GMT) |
---|---|---|
committer | Jouni Malinen <j@w1.fi> | 2014-12-08 09:42:07 (GMT) |
commit | a80ba67a26f51d61fc5875a22912c1316f1e1536 (patch) | |
tree | 1c5a113da32fe89ce7b1764dcfd245d9da81e867 /wpa_supplicant/config.c | |
parent | a9aaacbb5081f41318ddb0fa597da6c29c6898df (diff) | |
download | hostap-a80ba67a26f51d61fc5875a22912c1316f1e1536.zip hostap-a80ba67a26f51d61fc5875a22912c1316f1e1536.tar.gz hostap-a80ba67a26f51d61fc5875a22912c1316f1e1536.tar.bz2 |
Check os_snprintf() result more consistently - success case
This converts os_snprintf() result validation cases to use
os_snprintf_error() in cases where success condition was used to execute
a step. These changes were done automatically with spatch using the
following semantic patch:
@@
expression E1,E2,E3;
statement S1;
@@
E1 = os_snprintf(E2, E3, ...);
- if (\( E1 >= 0 \| E1 > 0 \) && \( (size_t) E1 < E3 \| E1 < (int) E3 \| E1 < E3 \))
+ if (!os_snprintf_error(E3, E1))
S1
Signed-off-by: Jouni Malinen <j@w1.fi>
Diffstat (limited to 'wpa_supplicant/config.c')
-rw-r--r-- | wpa_supplicant/config.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/wpa_supplicant/config.c b/wpa_supplicant/config.c index 5ebc1a8..91550c7 100644 --- a/wpa_supplicant/config.c +++ b/wpa_supplicant/config.c @@ -1310,7 +1310,7 @@ static int wpa_config_parse_wep_key(u8 *key, size_t *len, int line, os_memcpy(key, buf, *len); str_clear_free(buf); res = os_snprintf(title, sizeof(title), "wep_key%d", idx); - if (res >= 0 && (size_t) res < sizeof(title)) + if (!os_snprintf_error(sizeof(title), res)) wpa_hexdump_key(MSG_MSGDUMP, title, key, *len); return 0; } |