]> git.ipfire.org Git - thirdparty/hostap.git/commit - wpa_supplicant/config.c
Check os_snprintf() result more consistently - automatic 1
authorJouni Malinen <j@w1.fi>
Mon, 8 Dec 2014 09:15:51 +0000 (11:15 +0200)
committerJouni Malinen <j@w1.fi>
Mon, 8 Dec 2014 09:42:07 +0000 (11:42 +0200)
commitd85e1fc8a57d4736af7f5618bcccd86ad3345f94
treec5c895cefb076ed9925a7ed7b481532d198ddb26
parenta80ba67a26f51d61fc5875a22912c1316f1e1536
Check os_snprintf() result more consistently - automatic 1

This converts os_snprintf() result validation cases to use
os_snprintf_error() where the exact rule used in os_snprintf_error() was
used. These changes were done automatically with spatch using the
following semantic patch:

@@
identifier E1;
expression E2,E3,E4,E5,E6;
statement S1;
@@

(
  E1 = os_snprintf(E2, E3, ...);
|
  int E1 = os_snprintf(E2, E3, ...);
|
  if (E5)
E1 = os_snprintf(E2, E3, ...);
  else
E1 = os_snprintf(E2, E3, ...);
|
  if (E5)
E1 = os_snprintf(E2, E3, ...);
  else if (E6)
E1 = os_snprintf(E2, E3, ...);
  else
E1 = 0;
|
  if (E5) {
...
E1 = os_snprintf(E2, E3, ...);
  } else {
...
return -1;
  }
|
  if (E5) {
...
E1 = os_snprintf(E2, E3, ...);
  } else if (E6) {
...
E1 = os_snprintf(E2, E3, ...);
  } else {
...
return -1;
  }
|
  if (E5) {
...
E1 = os_snprintf(E2, E3, ...);
  } else {
...
E1 = os_snprintf(E2, E3, ...);
  }
)
? os_free(E4);
- if (E1 < 0 || \( E1 >= E3 \| (size_t) E1 >= E3 \| (unsigned int) E1 >= E3 \| E1 >= (int) E3 \))
+ if (os_snprintf_error(E3, E1))
(
  S1
|
{ ... }
)

Signed-off-by: Jouni Malinen <j@w1.fi>
49 files changed:
hostapd/ctrl_iface.c
hostapd/hlr_auc_gw.c
hostapd/hostapd_cli.c
src/ap/ctrl_iface_ap.c
src/ap/ieee802_1x.c
src/ap/wpa_auth.c
src/ap/wps_hostapd.c
src/common/wpa_common.c
src/common/wpa_ctrl.c
src/crypto/tls_openssl.c
src/drivers/driver_nl80211.c
src/drivers/driver_wext.c
src/eap_peer/eap.c
src/eap_peer/eap_fast.c
src/eap_peer/eap_fast_pac.c
src/eap_peer/eap_methods.c
src/eap_peer/eap_peap.c
src/eap_peer/eap_tls_common.c
src/eap_peer/eap_ttls.c
src/eap_server/eap_sim_db.c
src/eapol_auth/eapol_auth_dump.c
src/eapol_supp/eapol_supp_sm.c
src/p2p/p2p.c
src/p2p/p2p_parse.c
src/p2p/p2p_utils.c
src/radius/radius_server.c
src/rsn_supp/pmksa_cache.c
src/rsn_supp/preauth.c
src/rsn_supp/wpa.c
src/tls/asn1.c
src/tls/x509v3.c
src/utils/common.c
src/utils/uuid.c
src/utils/wpa_debug.c
src/wps/wps.c
src/wps/wps_common.c
src/wps/wps_registrar.c
wpa_supplicant/ap.c
wpa_supplicant/bgscan_learn.c
wpa_supplicant/config.c
wpa_supplicant/config_file.c
wpa_supplicant/ctrl_iface.c
wpa_supplicant/ctrl_iface_unix.c
wpa_supplicant/dbus/dbus_new_helpers.c
wpa_supplicant/eapol_test.c
wpa_supplicant/mesh.c
wpa_supplicant/wpa_cli.c
wpa_supplicant/wpas_glue.c
wpa_supplicant/wps_supplicant.c