Bug 339215 – networkmanager-0.8.1-r6: ifnet plugin causes NetworkManager crashed with segment fault!

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description cee1 2010-09-30 06:43:55 UTC]

networkmanager-0.8.1-r6: ifnet plugin causes NetworkManager crashed with segment fault!
I've removed ifnet plugin in a new ebuild(networkmanager-0.8.1-r7.ebuild), After installing the new ebuild, NM goes well, so it should be a bug of ifnet plugin.

Reproducible: Always

[Comment 1 cee1 2010-09-30 06:44:39 UTC]

Created attachment 249047 [details]
ebuild removed the ifnet plugin

[Comment 2 cee1 2010-09-30 06:48:02 UTC]

Created attachment 249048 [details]
/etc/conf.d/net -- with real values replaced by dummy ones

[Comment 3 Mu Qiao (RETIRED) 2010-10-01 05:55:25 UTC]

Thank you for reporting this. I'll try to make NM not crash for this case. But I think your configuration should be 
routes_eth0=( "default via 172.16.1.111" )
rather than routes_eth0=( "default gw 172.16.1.111" ). 
If you change "gw" to "via", I think NM won't crash.

[Comment 4 Xake 2010-11-14 08:25:06 UTC]

(In reply to comment #3)
> Thank you for reporting this. I'll try to make NM not crash for this case. But
> I think your configuration should be 
> routes_eth0=( "default via 172.16.1.111" )
> rather than routes_eth0=( "default gw 172.16.1.111" ). 
> If you change "gw" to "via", I think NM won't crash.
> 

you may want to support gw too, since that is old syntax and people may have missed that it has changed again, as it has before: bug 73641, comment 4.

[Comment 5 Mu Qiao (RETIRED) 2010-11-23 07:48:49 UTC]

Created attachment 255171 [details]
proposed ebuild

Hi dagger, I've fixed this issue @gitorious. So here is proposed ebuild. I'd love to take this bug if you think it's OK.

[Comment 6 Mu Qiao (RETIRED) 2010-11-23 07:49:17 UTC]

Created attachment 255173 [details]
the patch

here's the patch

[Comment 7 cee1 2010-11-23 08:03:25 UTC]

(In reply to comment #6)
> Created an attachment (id=255173) [details]
> the patch
> 
> here's the patch
> 
The plugin code was merged to upstream(http://cgit.freedesktop.org/NetworkManager/NetworkManager/commit/?id=38f3e5ca61cf8f66385ee45a442ff41a01f3885a).

Need commit this patch to upstream...

[Comment 8 Mu Qiao (RETIRED) 2010-11-23 09:17:32 UTC]

(In reply to comment #7)
> (In reply to comment #6)
> > Created an attachment (id=255173) [details] [details]
> > the patch
> > 
> > here's the patch
> > 
> The plugin code was merged to
> upstream(http://cgit.freedesktop.org/NetworkManager/NetworkManager/commit/?id=38f3e5ca61cf8f66385ee45a442ff41a01f3885a).
> 
> Need commit this patch to upstream...
> 

Yes, I will. But I'd like to have it tested before merging to upstream

[Comment 9 Mu Qiao (RETIRED) 2010-11-30 09:37:27 UTC]

networkmanager-0.8.2-r1 should fix this problem

[Comment 10 Mu Qiao (RETIRED) 2010-12-01 12:26:11 UTC]

Should be fixed now, close.