Am Sonntag, den 29.01.2006, 14:46 +0100 schrieb Phil Goss:
There seems to be a check in the gethostbyname for a NULL arg name ... if(!name) ... But no check for ... if(!strlen(name)) ....
Alexandre changed "strlen(xxx) > 0" in my first Patch to xxx[0] So for your code above, simple change:
if(!name) {
to
if((!name) && (name[0])) {
The logic is the same: use the code-path, if name is NULL and use it also, when the string is empty.
I send this type of patch yesterday and it's already in the tree.
The Commit-Message is here: http://www.winehq.org/pipermail/wine-cvs/2006-February/020657.html
(i used (pName != NULL) to have a common style in this function)
Does the above make sense, as I am not a programmer?
Yes, it does, and it's important when we have a Program that depend on this behaviour.
Not sure if this s bug in wine even though it implementation should be matching Windows native.
That's also a reason, why we have the Testsuite. A specific Test changes "should match" to "does not match (yet)" or "matches all the cases we tested".
The winedev-guide is nice to read. (see msg from James)