On Wed, 2014-07-02 at 11:26 +0200, Thomas Haller wrote: > On Tue, 2014-07-01 at 16:52 -0600, Chris Hessing wrote: > > Hi all, > > > I thought it might be an issue with the variant type I was sending in to > > some configuration setting that was later being parsed when I requested > > the connection become active. However, after going through my code to > > generate the settings, they all match what is documented here : > > https://developer.gnome.org/NetworkManager/0.9/ref-settings.html > > > > One interesting tidbit is that after Network Manager crashes, and is > > restarted by the system, the connection comes up fine and seems to work > > from then on. > > > > At this point, short of cracking open the Network Manager code, I am > > running out of ideas as to what it could be. Perhaps someone that > > understands the Network Manager code can shed some light on what might > > be going on? > > Hi Chris, > > > > This looks like a bug in NetworkManager. > > The attached patch should fix it.
Patch looks right to me. It's likely unnoticed because these days most certificates use the "path" scheme instead of the blob scheme, and thus this code doesn't get triggered. > Interestingly, > http://dbus.freedesktop.org/doc/dbus-glib/dbus-glib-Specializable-GType-System.html#DBUS-TYPE-G-UCHAR-ARRAY:CAPS > says that we cannot pass a GByteArray instead of the expected GArray of > uchar. But I think the comment is just wrong, because a GByteArray *is* > actually a GArray. We should look at this a bit more; I ran into this when doing the wwan-ipv6 stuff and it did cause a crash that was fixed by using GArray instead of GByteArray. But we've been using GByteArray for a really long time, so I'm sure that at some point in the past this worked. Dan _______________________________________________ networkmanager-list mailing list [email protected] https://mail.gnome.org/mailman/listinfo/networkmanager-list
