Mike Kaplinskiy <mike.kaplinskiy at gmail.com> writes: > As for the context note, it is perfectly valid code (segfault-less, > that is) as it stands, but we should either remove the null check on > the next line or assign the value later. No, the code is fine as it is. The bug is in cppcheck. -- Alexandre Julliard julliard at winehq.org