After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 743176 - [review PATCH] device: remember the method we've started ll addr conf with
[review PATCH] device: remember the method we've started ll addr conf with
Status: RESOLVED DUPLICATE of bug 724041
Product: NetworkManager
Classification: Platform
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: NetworkManager maintainer(s)
NetworkManager maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2015-01-19 12:34 UTC by Lubomir Rintel
Modified: 2015-02-25 14:09 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
device: remember the method we've started ll addr conf with (3.26 KB, text/plain)
2015-01-19 12:34 UTC, Lubomir Rintel 		Details

Description Lubomir Rintel 2015-01-19 12:34:35 UTC 
Created attachment 294871 [details]
device: remember the method we've started ll addr conf with

User can change the connection in between and hit an assert in supposedly
    unreachable code in linklocal6_complete():
    
      # nmcli c up CON & nmcli c modify CON ipv6.method ignore
      ...
      NetworkManager[18134]: linklocal6_complete: assertion 'FALSE' failed
Comment 1 Thomas Haller 2015-01-19 14:10:59 UTC 
this is a special case of bug 724041 .
Comment 2 Dan Williams 2015-01-19 18:52:19 UTC 
LGTM
Comment 3 Thomas Haller 2015-01-19 21:35:53 UTC 
I don't think that this is the right fix, because there are many situations that can fail in a similar way. Making an individual copy of each property seems not the way to go.
Comment 4 Lubomir Rintel 2015-02-07 16:44:52 UTC 

*** This bug has been marked as a duplicate of bug 724041 ***