[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [bug #19225] bind, connect, and sendto should verify input
From: |
Dmitry Potapov |
Subject: |
[lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers |
Date: |
Wed, 07 Mar 2007 09:52:32 +0000 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.10) Gecko/20070216 Firefox/1.5.0.10 |
Follow-up Comment #3, bug #19225 (project lwip):
I believe that the whole socket.c is overhead that you are ready to pay for
better compatibility with standard sockets. So, if you worry about footprint
and performance, it is better to use netconn_ interface. And, we check the
first parameter - socket descriptor, why don't have a reasonable check for
others? If you think it adds big overhead, then let's make it ASSERT.
BTW, there is no need to use set_errno(EBADF) if get_socket(s) returns NULL,
because get_socket has already set errno.
sock = get_socket(s);
if (!sock) {
- set_errno(EBADF);
return -1;
}
As to api_lib.c, I prefer to see ASSERT instead of that not NULL checking.
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/bugs/?19225>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Dmitry Potapov, 2007/03/07
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Kieran Mansley, 2007/03/07
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Frédéric Bernon, 2007/03/07
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers,
Dmitry Potapov <=
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Frédéric Bernon, 2007/03/07
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Simon Goldschmidt, 2007/03/07
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Frédéric Bernon, 2007/03/07
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Simon Goldschmidt, 2007/03/07
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Kieran Mansley, 2007/03/07
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Dmitry Potapov, 2007/03/07
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Frédéric Bernon, 2007/03/11
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Frédéric Bernon, 2007/03/11
- [lwip-devel] [bug #19225] bind, connect, and sendto should verify input paramers, Frédéric Bernon, 2007/03/11