Messages in this thread | | | Date | Fri, 23 May 2014 09:00:16 +0200 | From | Jesper Nilsson <> | Subject | Re: [PATCH] arch: cris: arch-v32: drivers: axisflashmap.c: Cleaning up inconsistent NULL checks |
| |
On Fri, May 23, 2014 at 12:00:09AM +0200, Rickard Strandqvist wrote: > Cleaning up inconsistent NULL checks. > There is otherwise a risk of a possible null pointer dereference. > > Was largely found by using a static code analysis program called cppcheck.
... and is a false positive.
We can't enter this if-block unless ptable_head is set, which is only set if main_mtd is set.
It might be useful to add it anyways, but there's a refactoring needed for the axisflashmaps, so I'll keep it for then.
/^JN - Jesper Nilsson -- Jesper Nilsson -- jesper.nilsson@axis.com
| |