On Fri, 10 Jun 2011 14:19:43 -0700, Juan Lang wrote:
This change doesn't accomplish anything.
Yes it does. It makes Clang happy and avoids a null pointer dereference in case the tests are failing. (Even if they're passing now, someone might break them in future.)
In the first place, the previous ok which you did not change already ensures pdst is not NULL.
Yes, I'm sure it does, but that has really nothing to do with this. The next line is run in either case, and if pdst is null, it will cause a segfault.
Second, the ok output message also dereferences pdst by printing it
That's not true. "%s" formats null pointers as "(null)" instead of dereferencing anything. Feel free to try it yourself or see ntdll/printf.c:225 for such a check.
Really, I suggest you just ignore this warning, it's not worth the extra baggage.
It would be a lot easier to find and fix real things if there weren't a thousand false ones hanging around. But if you feel it's better to have a lot of warnings and some possible bugs than a lot of checks and no bugs, then maybe I'll not waste any more time "fixing" them.