Hello Raghu Vatsavayi,
The patch f21fb3ed364b: "Add support of Cavium Liquidio ethernet
adapters" from Jun 9, 2015, leads to the following static checker
warning:
drivers/net/ethernet/cavium/liquidio/octeon_console.c:326 cvmx_bootmem_phy_named_block_find()
error: potential null dereference 'name_tmp'. (kmalloc returns null)
drivers/net/ethernet/cavium/liquidio/octeon_console.c
319 for (i = 0; i < num_blocks; i++) {
320 u64 named_size =
321 CVMX_BOOTMEM_NAMED_GET_FIELD(oct, named_addr,
322 size);
323 if (name && named_size) {
324 char *name_tmp =
325 kmalloc(name_length + 1, GFP_KERNEL);
kmalloc can fail.
326 CVMX_BOOTMEM_NAMED_GET_NAME(oct, named_addr,
327 name_tmp,
328 name_length);
329 if (!strncmp(name, name_tmp, name_length)) {
330 result = named_addr;
331 kfree(name_tmp);
332 break;
333 }
334 kfree(name_tmp);
335 } else if (!name && !named_size) {
336 result = named_addr;
337 break;
338 }
339
340 named_addr +=
341 sizeof(struct cvmx_bootmem_named_block_desc);
342 }
regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
