lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 11 Apr 2016 12:00:04 +0200 From: Christophe JAILLET <christophe.jaillet@...adoo.fr> To: geoff@...radead.org, netdev@...r.kernel.org, linuxppc-dev@...ts.ozlabs.org, kernel-janitors@...r.kernel.org Subject: Inconsistent use of size argument in kzalloc and memcpy in 'drivers/net/ethernet/toshiba/ps3_gelic_wireless.c' Hi, while looking at potential clean-up, I ended on the following code which looks spurious to me. We allocate 'be16_to_cpu(scan_info->size)' bytes, but then copy 'scan_info->size'. This is not consistent. I don't know which one is the correct one. CJ --- drivers/net/ethernet/toshiba/ps3_gelic_wireless.c +++ /tmp/cocci-output-24201-0dddbd-ps3_gelic_wireless.c @@ -1616,13 +1616,10 @@ static void gelic_wl_scan_complete_event target->valid = 1; target->eurus_index = i; kfree(target->hwinfo); - target->hwinfo = kzalloc(be16_to_cpu(scan_info->size), - GFP_KERNEL); if (!target->hwinfo) continue; /* copy hw scan info */ - memcpy(target->hwinfo, scan_info, scan_info->size); target->essid_len = strnlen(scan_info->essid, sizeof(scan_info->essid)); target->rate_len = 0;
Powered by blists - more mailing lists