netdev - IPv6: Removing unnecessary NULL checks patch causes crash.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:	Wed, 16 Nov 2011 11:51:59 +0200
From:	Matti Vaittinen <matti.vaittinen@....com>
To:	davem@...emloft.net
Cc:	netdev@...r.kernel.org
Subject: IPv6: Removing unnecessary NULL checks patch causes crash.

I left my box to run some test scripts last night with net-next tree where patch
IPv6: Removing unnecessary NULL checks was applied. Crash had occurred when script 
executed "ip route change xxx/128 dev eth0" with non existing target twice 
in a row. (non existing target == there was no previous route to be changed) 

This was not occuring with kernel where only 
IPv6 routing, NLM_F_* flag support: REPLACE and EXCL flags support, warn about missing CREATE flag
and
IPv6 routing, NLM_F_* flag support: warn if new route is created without NLM_F_CREATE
were applied. 

Anyways, I cannot think how info struct could be NULL, so I am a bit
puzzled. Looks like I am missing some piece here. Anyways, somewhere in
these patches is lurking a bug. I try to find the reason. 

Sorry for the trouble.

--Matti

-- 
Matti Vaittinen
+358 504863070
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Told a UDP joke the other night...
...but I'm not sure everyone got it...
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html