KIO::http_post bug fixed

[Christian Weilbach]

Hi,

as actually nobody helped me, I've helped myself and have fixed another 
http_post bug. This one is in the http.cpp file and is about header parsing.

As Thiago has pointed out in IRC kDebug( xxxx ) does not work, so I have 
replaced it by kDebug(). Otherwise it will only throw the header, which might 
be o.k., but was to less for me.

The actual bugfix is the part about metaData( "content-length" ), which is not 
checked for its length correctly and therefore may introduce one carriage 
return to much, therefore the "Content-length: xxx" is not regarded as part 
of the header by the server.

Another bug fixed imho is the replacement of isNull() with isEmpty(), actually 
there are many accurances of isNull() in http.cpp, which in my opinion should 
be replaced as much as possible with isEmpty(). Another bug fixed by Antonio 
had the same problem.

Cheers and I hope you can fix it upstream soon,
Christian

-------------- next part --------------
A non-text attachment was scrubbed...
Name: kio_content_length_patch.diff
Type: text/x-diff
Size: 46404 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20070722/bfbb3b6c/attachment.diff>