Anton Arapov wrote: > "Denis V. Lunev" <[EMAIL PROTECTED]> writes: >> Anton Arapov wrote: >>> "Denis V. Lunev" <[EMAIL PROTECTED]> writes: >>>> This code is broken from the very beginning. >>>> >>>> iris den # cat /proc/sys/net/ipv4/ip_local_port_range >>>> 32768 61000 >>>> iris den # echo 32768 32 >/proc/sys/net/ipv4/ip_local_port_range >>>> iris den # cat /proc/sys/net/ipv4/ip_local_port_range >>>> 32768 32 >>>> iris den # echo 32768 61000 >/proc/sys/net/ipv4/ip_local_port_range >>> If you're talking about checks in sysctl, I believe it should be >>> another patch for sysctl only, and I'm going to push it via -mm tree. >>> >>> the devision by zero exists in inet_connection_socket.c, and must be >>> fixed for sure because the situation with the same min and max port >>> numbers in sysctl are possible and not prohibited. >>> >>> Cheers! >> your patch change nothing :( unfortunately. If I set '32768 32767' it >> will oops again. > > Patch prevents the system crash. System traps on division by zero. > > Your case(MAX<MIN) must be handled by sysctl. Do you want to say > that I have to join patch for sysctl.c to this one? It's bad idea. >
both versions of settings, your ones and my ones are _useless_ in real life. So, we do some sanity fixes. Am I right? If so, we must prevent all versions of OOPS (aka division by zero here). I'll send my vision in a moment... Regards, Den - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html