Fix more wrong sizeof() usages, part 4/x
authorSascha Wildner <saw@online.de>
Sat, 30 Jun 2012 08:44:33 +0000 (10:44 +0200)
committerSascha Wildner <saw@online.de>
Sat, 30 Jun 2012 08:44:46 +0000 (10:44 +0200)
We want struct sizes here, not pointer sizes.

Found-with: Coccinelle (http://coccinelle.lip6.fr/)

contrib/hostapd/src/crypto/md5.c
contrib/hostapd/src/drivers/driver_ralink.c
contrib/wpa_supplicant/src/crypto/md5.c
contrib/wpa_supplicant/src/drivers/driver_ralink.c

index a7db7aa..e13ce7d 100644 (file)
@@ -287,7 +287,7 @@ void MD5Final(unsigned char digest[16], struct MD5Context *ctx)
     MD5Transform(ctx->buf, (u32 *) ctx->in);
     byteReverse((unsigned char *) ctx->buf, 4);
     os_memcpy(digest, ctx->buf, 16);
-    os_memset(ctx, 0, sizeof(ctx));    /* In case it's sensitive */
+    os_memset(ctx, 0, sizeof(*ctx));   /* In case it's sensitive */
 }
 
 /* The four core functions - F1 is optimized somewhat */
index e9313cb..e14e70b 100644 (file)
@@ -232,7 +232,7 @@ static int wpa_driver_ralink_set_ssid(struct wpa_driver_ralink_data *drv,
        buf = os_zalloc(sizeof(NDIS_802_11_SSID));
        if (buf == NULL)
                return -1;
-       os_memset(buf, 0, sizeof(buf));
+       os_memset(buf, 0, sizeof(*buf));
        buf->SsidLength = ssid_len;
        os_memcpy(buf->Ssid, ssid, ssid_len);
        os_memset(&iwr, 0, sizeof(iwr));
index a7db7aa..e13ce7d 100644 (file)
@@ -287,7 +287,7 @@ void MD5Final(unsigned char digest[16], struct MD5Context *ctx)
     MD5Transform(ctx->buf, (u32 *) ctx->in);
     byteReverse((unsigned char *) ctx->buf, 4);
     os_memcpy(digest, ctx->buf, 16);
-    os_memset(ctx, 0, sizeof(ctx));    /* In case it's sensitive */
+    os_memset(ctx, 0, sizeof(*ctx));   /* In case it's sensitive */
 }
 
 /* The four core functions - F1 is optimized somewhat */
index e9313cb..e14e70b 100644 (file)
@@ -232,7 +232,7 @@ static int wpa_driver_ralink_set_ssid(struct wpa_driver_ralink_data *drv,
        buf = os_zalloc(sizeof(NDIS_802_11_SSID));
        if (buf == NULL)
                return -1;
-       os_memset(buf, 0, sizeof(buf));
+       os_memset(buf, 0, sizeof(*buf));
        buf->SsidLength = ssid_len;
        os_memcpy(buf->Ssid, ssid, ssid_len);
        os_memset(&iwr, 0, sizeof(iwr));