lp:~lkundrak/network-manager/launchpad-master

Branch merges

This import branch has no branches proposed for merge into it.

No branches dependent on this one.

Related source package recipes

1 recipe using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

12336. By Thomas Haller on 2018-04-18

auth-manager: fix potential issue iterating modified CList in _dbus_new_proxy_cb()

In the loop, we invoke callbacks. What the callbacks do, is out of control
of NMAuthManager. For example, they could cancel or schedule new
requests. Especially, cancelling invalidate the stored @safe pointer.

Fix that, by always iterate from the start of the list.

Fixes: d0563f0733ed293d67e9a0f6503e28c3f1c08f1b

12335. By Beniamino Galvani <email address hidden> on 2018-04-17

auth-manager: fix processing calls in _dbus_new_proxy_cb()

In the first loop, the element is removed only when the callback is
executed. The second loop never removes the current element. Use the
for_each macro for both.

Fixes: d0563f0733ed293d67e9a0f6503e28c3f1c08f1b

12334. By Beniamino Galvani <email address hidden> on 2018-04-17

core: fix wrong assertion when disposing NMAuthManager

The list should be empty on disposal.

Fixes: 2ea2df3184d45567fa9c44f5ef90634a779bfb75

12333. By Richard Schütz on 2018-04-17

dhcp: don't enforce broadcast flag

Requesting broadcast replies from the DHCP server can be problematic in
filtered environments like some wireless networks. Don't override the
default of using unicast. This matches the behaviour of the external DHCP
clients.

https://github.com/NetworkManager/NetworkManager/pull/93

12332. By Beniamino Galvani <email address hidden> on 2018-04-17

auth-manager: don't process idle calls when the proxy creation finishes

The list of calls contains two kinds of elements: (1) calls that don't
need a D-Bus request and are only waiting for the asynchronous
invocation of the callback in an idle function; (2) calls that need a
D-Bus request and are waiting for the D-Bus proxy.

When the proxy creation finishes, only (2) calls must be canceled (if
the creation failed) or started (if the proxy was created).

Fixes: 798b2a7527bddadcec37b48183da313fbc961e45

https://bugzilla.redhat.com/show_bug.cgi?id=1567807

12331. By Thomas Haller on 2018-04-16

cli/polkit: merge branch 'th/polkit-cleanup'

https://github.com/NetworkManager/NetworkManager/pull/90

12330. By Thomas Haller on 2018-04-16

all: merge branch 'th/settings-flags-rh1460295'

https://bugzilla.redhat.com/show_bug.cgi?id=1460295
https://github.com/NetworkManager/NetworkManager/pull/88

12329. By Beniamino Galvani <email address hidden> on 2018-04-13

cli: merge branch 'bg/nmcli-overview-rh1434527'

https://bugzilla.redhat.com/show_bug.cgi?id=1434527

12328. By Thomas Haller on 2018-04-13

auth-subject: fix potential memory corruption in nm_auth_subject_to_string()

We don't want to apped the value to @buf, we want to set it.
Also, if @buf happens to be uninitialized, g_strlcat() might
determine there is nothing to append and return the buffer unmodified.
Then, the (non NULL terminated) buffer might be printed.

Note that before recent refactoring, we effectively would only call
nm_auth_subject_to_string() on auth-subjects that were of type
UNIX-PROCESS. Hence, this bug came only to light very recently,
although it was present for a long time.

Fixes: eabe7d856c243673bbaba3295ce74d72e188596d

12327. By Thomas Haller on 2018-04-13

policy: fix potential leak of subject in auto_activate_device()