Socket.c bugs

Question

I found two issues so far with socket.c source code my file version is: <2014/05/01> V1.0.3. Refer to M20140501 sock_io_mode issue: I was expecting that this flag be cleaned when you close the socket. The other issue is related with recv In block io mode, it doesn’t return until data reception is completed - data is filled as len in socket buffer. but what is implemented is that it will return as soon as it get any data: if(recvsize != 0) break; I think this code need review.

kammutierspule · Answer

I was looking for BSD recv function and It looks that your implementation is correct: linux.die.net/man/2/recv “The receive calls normally return any data available, up to the requested amount, rather than waiting for receipt of the full amount requested.” So the issue is in the wiznet socket.h documentation: [quote] It waits for data as much as the application wants to receive. … In block io mode, it doesn’t return until data reception is completed - data is filled as len in socket buffer.[/quote] So, it no correct to say “data is filled as len in socket buffer”

alexpescaru · Answer

And I believe there is one more thing to correct in socket.c, function sendto(). Bellow, I believe should be taddr, not addr. Am I right? //M20140501 : For avoiding fatal error on memory align mismatched //if(((uint32_t)addr) == 0) return SOCKERR_IPINVALID; { uint32_t taddr; taddr = ((uint32_t)addr[0]) & 0x000000FF; taddr = (taddr << 8) + ((uint32_t)addr[1] & 0x000000FF); taddr = (taddr << 8) + ((uint32_t)addr[2] & 0x000000FF); taddr = (taddr << 8) + ((uint32_t)addr[3] & 0x000000FF); } // if(((uint32_t)[color=#FF0000]addr[/color]) == 0) return SOCKERR_IPINVALID;

kammutierspule · Answer

Wouldn’t it be better if this ioLibrary_BSD was in a public repository (github wiznet account?) so we all can use the appropriated tools and platform for bug reporting, version control… etc?!

fherbert · Answer

I’ve just checked the current version (1.0.3) of code for socket.c at [url]https://github.com/Wiznet/W5500_EVB[/url] or [url]http://wizwiki.net/wiki/doku.php?id=products:w5500:driver[/url] and the fixes don’t appear to have made it to this driver yet. When will this driver be updated at github?

hjjeon0608 · Answer

Hi, fherbert We are modifying code about your point and other points. So, we will update next version ASAP! Plz, wait a while kindly~:D