Line 474 of drivers/parisc/dino.c reads
res->name = dino_dev->hba.lmmio_space.name;
but res is &dino_dev->hba.lmmio_space, so this seems to just assign NULL
again to res->name. Presumably the allocation of <= 128 bytes never
fails, but I wonder what the purpose was, and if the confusing else
branch couldn't just be deleted (and then one should just do res->name =
kmemdup(name, size+1, GFP_KERNEL); and delete the if as well).
Rasmus
--
To unsubscribe from this list: send the line "unsubscribe linux-parisc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
