[Bug 22514] lstrlen is implemented incorrectly (dlls/kernel32/string.c)

[wine-bugs at winehq.org]

http://bugs.winehq.org/show_bug.cgi?id=22514





--- Comment #13 from Dmitry Timoshkov <dmitry at codeweavers.com>  2010-04-28 10:20:15 ---
(In reply to comment #12)
> Chromium uses the documented "If lpString is NULL, the return value is 0"
> feature.

For what? Wine perfectly does that as well.

> What Valgrind is complaining about is reading *(NULL).
> I know you do it in the try-statement but
> a) I don't think it's a good idea to handle this case using exceptions

Windows does that in an exception handler, Wine must follow that.

> b) the native implementation doesn't call SetLastError while Wine does

Please feel free to send a patch with a test case to wine-patches.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.