Update to 1.14.2 release

This commit is contained in:
Lubomir Rintel 2018-10-19 16:58:06 +02:00
parent 92d36d7fb0
commit 6fd65f9fd5
4 changed files with 122 additions and 545 deletions

1
.gitignore vendored
View File

@ -349,3 +349,4 @@ network-manager-applet-0.8.1.tar.bz2
/NetworkManager-1.12.0.tar.xz
/NetworkManager-1.12.2.tar.xz
/NetworkManager-1.14.0.tar.xz
/NetworkManager-1.14.2.tar.xz

View File

@ -1,537 +0,0 @@
From 545851463bcaed25a80eecb864f91c23eec9cf7a Mon Sep 17 00:00:00 2001
From: Thomas Haller <thaller@redhat.com>
Date: Mon, 17 Sep 2018 11:08:15 +0200
Subject: [PATCH 1/1] connectivity: fix crash when removing easy-handle from
curl callback
libcurl does not allow removing easy-handles from within a curl
callback.
That was already partly avoided for one handle alone. That is, when
a handle completed inside a libcurl callback, it would only invoke the
callback, but not yet delete it. However, that is not enough, because
from within a callback another handle can be cancelled, leading to
the removal of (the other) handle and a crash:
==24572== at 0x40319AB: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==24572== by 0x52DDAE5: Curl_close (url.c:392)
==24572== by 0x52EC02C: curl_easy_cleanup (easy.c:825)
==24572== by 0x5FDCD2: cb_data_free (nm-connectivity.c:215)
==24572== by 0x5FF6DE: nm_connectivity_check_cancel (nm-connectivity.c:585)
==24572== by 0x55F7F9: concheck_handle_complete (nm-device.c:2601)
==24572== by 0x574C12: concheck_cb (nm-device.c:2725)
==24572== by 0x5FD887: cb_data_invoke_callback (nm-connectivity.c:167)
==24572== by 0x5FD959: easy_header_cb (nm-connectivity.c:435)
==24572== by 0x52D73CB: chop_write (sendf.c:612)
==24572== by 0x52D73CB: Curl_client_write (sendf.c:668)
==24572== by 0x52D54ED: Curl_http_readwrite_headers (http.c:3904)
==24572== by 0x52E9EA7: readwrite_data (transfer.c:548)
==24572== by 0x52E9EA7: Curl_readwrite (transfer.c:1161)
==24572== by 0x52F4193: multi_runsingle (multi.c:1915)
==24572== by 0x52F5531: multi_socket (multi.c:2607)
==24572== by 0x52F5804: curl_multi_socket_action (multi.c:2771)
Fix that, by never invoking any callbacks when we are inside a libcurl
callback. Instead, the handle is marked for completion and queued. Later,
we complete all queue handles separately.
While at it, drop the @error argument from NMConnectivityCheckCallback.
It was only used to signal cancellation. Let's instead signal that via
status NM_CONNECTIVITY_CANCELLED.
https://bugzilla.gnome.org/show_bug.cgi?id=797136
https://bugs.launchpad.net/ubuntu/+source/network-manager/+bug/1792745
https://bugzilla.opensuse.org/show_bug.cgi?id=1107197
https://github.com/NetworkManager/NetworkManager/pull/207
Fixes: d8a31794c8b9db243076ba0c24dfe6e496b78697
(cherry picked from commit fa40fc6d765248194d692a090b3abfbea6fe4e8f)
(cherry picked from commit 7f05debf99ab30227180acd7b24a363013c957f2)
---
src/devices/nm-device.c | 14 +--
src/nm-connectivity.c | 236 ++++++++++++++++++++++++----------------
src/nm-connectivity.h | 7 +-
3 files changed, 153 insertions(+), 104 deletions(-)
diff --git a/src/devices/nm-device.c b/src/devices/nm-device.c
index 8ca899a28..5ed4e559b 100644
--- a/src/devices/nm-device.c
+++ b/src/devices/nm-device.c
@@ -2813,7 +2813,6 @@ static void
concheck_cb (NMConnectivity *connectivity,
NMConnectivityCheckHandle *c_handle,
NMConnectivityState state,
- GError *error,
gpointer user_data)
{
_nm_unused gs_unref_object NMDevice *self_keep_alive = NULL;
@@ -2834,7 +2833,7 @@ concheck_cb (NMConnectivity *connectivity,
handle->c_handle = NULL;
self = handle->self;
- if (nm_utils_error_is_cancelled (error, FALSE)) {
+ if (state == NM_CONNECTIVITY_CANCELLED) {
/* the only place where we nm_connectivity_check_cancel(@c_handle), is
* from inside concheck_handle_complete(). This is a recursive call,
* nothing to do. */
@@ -2843,15 +2842,14 @@ concheck_cb (NMConnectivity *connectivity,
return;
}
+ /* we keep NMConnectivity instance alive. It cannot be disposing. */
+ nm_assert (state != NM_CONNECTIVITY_DISPOSING);
+
self_keep_alive = g_object_ref (self);
- _LOGT (LOGD_CONCHECK, "connectivity: complete check (seq:%llu, state:%s%s%s%s)",
+ _LOGT (LOGD_CONCHECK, "connectivity: complete check (seq:%llu, state:%s)",
(long long unsigned) handle->seq,
- nm_connectivity_state_to_string (state),
- NM_PRINT_FMT_QUOTED (error, ", error: ", error->message, "", ""));
-
- /* we keep NMConnectivity instance alive. It cannot be disposing. */
- nm_assert (!nm_utils_error_is_cancelled (error, TRUE));
+ nm_connectivity_state_to_string (state));
/* keep @self alive, while we invoke callbacks. */
priv = NM_DEVICE_GET_PRIVATE (self);
diff --git a/src/nm-connectivity.c b/src/nm-connectivity.c
index 3b779677d..8a6e955ae 100644
--- a/src/nm-connectivity.c
+++ b/src/nm-connectivity.c
@@ -46,8 +46,10 @@ NM_UTILS_LOOKUP_STR_DEFINE_STATIC (_state_to_string, int /*NMConnectivityState*/
NM_UTILS_LOOKUP_STR_ITEM (NM_CONNECTIVITY_PORTAL, "PORTAL"),
NM_UTILS_LOOKUP_STR_ITEM (NM_CONNECTIVITY_FULL, "FULL"),
- NM_UTILS_LOOKUP_STR_ITEM (NM_CONNECTIVITY_ERROR, "ERROR"),
- NM_UTILS_LOOKUP_STR_ITEM (NM_CONNECTIVITY_FAKE, "FAKE"),
+ NM_UTILS_LOOKUP_STR_ITEM (NM_CONNECTIVITY_ERROR, "ERROR"),
+ NM_UTILS_LOOKUP_STR_ITEM (NM_CONNECTIVITY_FAKE, "FAKE"),
+ NM_UTILS_LOOKUP_STR_ITEM (NM_CONNECTIVITY_CANCELLED, "CANCELLED"),
+ NM_UTILS_LOOKUP_STR_ITEM (NM_CONNECTIVITY_DISPOSING, "DISPOSING"),
);
const char *
@@ -77,6 +79,10 @@ struct _NMConnectivityCheckHandle {
} concheck;
#endif
+ const char *completed_log_message;
+ char *completed_log_message_free;
+ NMConnectivityState completed_state;
+
guint timeout_id;
};
@@ -90,6 +96,7 @@ static guint signals[LAST_SIGNAL] = { 0 };
typedef struct {
CList handles_lst_head;
+ CList completed_handles_lst_head;
char *uri;
char *response;
gboolean enabled;
@@ -142,60 +149,37 @@ NM_DEFINE_SINGLETON_GETTER (NMConnectivity, nm_connectivity_get, NM_TYPE_CONNECT
/*****************************************************************************/
static void
-cb_data_invoke_callback (NMConnectivityCheckHandle *cb_data,
- NMConnectivityState state,
- GError *error,
- const char *log_message)
+cb_data_complete (NMConnectivityCheckHandle *cb_data,
+ NMConnectivityState state,
+ const char *log_message)
{
- NMConnectivityCheckCallback callback;
+ NMConnectivity *self;
nm_assert (cb_data);
nm_assert (NM_IS_CONNECTIVITY (cb_data->self));
-
- callback = cb_data->callback;
- if (!callback)
- return;
-
- cb_data->callback = NULL;
-
+ nm_assert (cb_data->callback);
+ nm_assert (state != NM_CONNECTIVITY_UNKNOWN);
nm_assert (log_message);
- _LOG2D ("check completed: %s; %s",
- nm_connectivity_state_to_string (state),
- log_message);
-
- callback (cb_data->self,
- cb_data,
- state,
- error,
- cb_data->user_data);
-}
-
-static void
-cb_data_free (NMConnectivityCheckHandle *cb_data,
- NMConnectivityState state,
- GError *error,
- const char *log_message)
-{
- NMConnectivity *self;
-
- nm_assert (cb_data);
-
self = cb_data->self;
- nm_assert (NM_IS_CONNECTIVITY (self));
+ /* mark the handle as completing. After this point, nm_connectivity_check_cancel()
+ * is no longer possible. */
+ cb_data->self = NULL;
- c_list_unlink (&cb_data->handles_lst);
+ c_list_unlink_stale (&cb_data->handles_lst);
#if WITH_CONCHECK
if (cb_data->concheck.curl_ehandle) {
NMConnectivityPrivate *priv;
/* Contrary to what cURL manual claim it is *not* safe to remove
- * the easy handle "at any moment"; specifically not from the
- * write function. Thus here we just dissociate the cb_data from
- * the easy handle and the easy handle will be cleaned up when the
- * message goes to CURLMSG_DONE in _con_curl_check_connectivity(). */
+ * the easy handle "at any moment"; specifically it's not safe to
+ * remove *any* handle from within a libcurl callback. That is
+ * why we queue completed handles in this case.
+ *
+ * cb_data_complete() is however only called *not* from within a
+ * libcurl callback. So, this is fine. */
curl_easy_setopt (cb_data->concheck.curl_ehandle, CURLOPT_WRITEFUNCTION, NULL);
curl_easy_setopt (cb_data->concheck.curl_ehandle, CURLOPT_WRITEDATA, NULL);
curl_easy_setopt (cb_data->concheck.curl_ehandle, CURLOPT_HEADERFUNCTION, NULL);
@@ -214,7 +198,18 @@ cb_data_free (NMConnectivityCheckHandle *cb_data,
nm_clear_g_source (&cb_data->timeout_id);
- cb_data_invoke_callback (cb_data, state, error, log_message);
+ _LOG2D ("check completed: %s; %s",
+ nm_connectivity_state_to_string (state),
+ log_message);
+
+ cb_data->callback (self,
+ cb_data,
+ state,
+ cb_data->user_data);
+
+ /* Note: self might be a danling pointer at this point. It must not be used
+ * after this point, and all callers must either take a reference first, or
+ * not use the self pointer too. */
#if WITH_CONCHECK
g_free (cb_data->concheck.response);
@@ -222,12 +217,54 @@ cb_data_free (NMConnectivityCheckHandle *cb_data,
g_string_free (cb_data->concheck.recv_msg, TRUE);
#endif
g_free (cb_data->ifspec);
+ if (cb_data->completed_log_message_free)
+ g_free (cb_data->completed_log_message_free);
g_slice_free (NMConnectivityCheckHandle, cb_data);
}
/*****************************************************************************/
#if WITH_CONCHECK
+
+static void
+cb_data_queue_completed (NMConnectivityCheckHandle *cb_data,
+ NMConnectivityState state,
+ const char *log_message_static,
+ char *log_message_take /* take */)
+{
+ nm_assert (cb_data);
+ nm_assert (NM_IS_CONNECTIVITY (cb_data->self));
+ nm_assert (state != NM_CONNECTIVITY_UNKNOWN);
+ nm_assert (log_message_static || log_message_take);
+ nm_assert (cb_data->completed_state == NM_CONNECTIVITY_UNKNOWN);
+ nm_assert (!cb_data->completed_log_message);
+ nm_assert (c_list_contains (&NM_CONNECTIVITY_GET_PRIVATE (cb_data->self)->handles_lst_head, &cb_data->handles_lst));
+
+ cb_data->completed_state = state;
+ cb_data->completed_log_message = log_message_static ?: log_message_take;
+ cb_data->completed_log_message_free = log_message_take;
+
+ c_list_unlink_stale (&cb_data->handles_lst);
+ c_list_link_tail (&NM_CONNECTIVITY_GET_PRIVATE (cb_data->self)->completed_handles_lst_head, &cb_data->handles_lst);
+}
+
+static void
+_complete_queued (NMConnectivity *self)
+{
+ NMConnectivity *self_keep_alive = NULL;
+ NMConnectivityPrivate *priv = NM_CONNECTIVITY_GET_PRIVATE (self);
+ NMConnectivityCheckHandle *cb_data;
+
+ while ((cb_data = c_list_first_entry (&priv->completed_handles_lst_head, NMConnectivityCheckHandle, handles_lst))) {
+ if (!self_keep_alive)
+ self_keep_alive = g_object_ref (self);
+ cb_data_complete (cb_data,
+ cb_data->completed_state,
+ cb_data->completed_log_message);
+ }
+ nm_g_object_unref (self_keep_alive);
+}
+
static const char *
_check_handle_get_response (NMConnectivityCheckHandle *cb_data)
{
@@ -265,29 +302,37 @@ _con_curl_check_connectivity (CURLM *mhandle, int sockfd, int ev_bitmask)
continue;
}
- if (!cb_data->callback) {
- /* callback was already invoked earlier. */
- cb_data_free (cb_data, NM_CONNECTIVITY_UNKNOWN, NULL, NULL);
- } else if (msg->data.result != CURLE_OK) {
- gs_free char *log_message = NULL;
+ nm_assert (cb_data);
+ nm_assert (NM_IS_CONNECTIVITY (cb_data->self));
- log_message = g_strdup_printf ("check failed with curl status %d", msg->data.result);
- cb_data_free (cb_data, NM_CONNECTIVITY_LIMITED, NULL,
- log_message);
+ if (cb_data->completed_state != NM_CONNECTIVITY_UNKNOWN) {
+ /* callback was already invoked earlier. Nothing to do. */
+ continue;
+ }
+
+ if (msg->data.result != CURLE_OK) {
+ cb_data_queue_completed (cb_data,
+ NM_CONNECTIVITY_LIMITED,
+ NULL,
+ g_strdup_printf ("check failed with curl status %d", msg->data.result));
} else if ( !((_check_handle_get_response (cb_data))[0])
&& (curl_easy_getinfo (msg->easy_handle, CURLINFO_RESPONSE_CODE, &response_code) == CURLE_OK)
&& response_code == 204) {
/* If we got a 204 response code (no content) and we actually
* requested no content, report full connectivity. */
- cb_data_free (cb_data, NM_CONNECTIVITY_FULL, NULL,
- "no content, as expected");
+ cb_data_queue_completed (cb_data,
+ NM_CONNECTIVITY_FULL,
+ "no content, as expected",
+ NULL);
} else {
/* If we get here, it means that easy_write_cb() didn't read enough
* bytes to be able to do a match, or that we were asking for no content
* (204 response code) and we actually got some. Either way, that is
* an indication of a captive portal */
- cb_data_free (cb_data, NM_CONNECTIVITY_PORTAL, NULL,
- "unexpected short response");
+ cb_data_queue_completed (cb_data,
+ NM_CONNECTIVITY_PORTAL,
+ "unexpected short response",
+ NULL);
}
}
@@ -306,6 +351,7 @@ _con_curl_timeout_cb (gpointer user_data)
priv->concheck.curl_timer = 0;
_con_curl_check_connectivity (priv->concheck.curl_mhandle, CURL_SOCKET_TIMEOUT, 0);
+ _complete_queued (self);
return G_SOURCE_REMOVE;
}
@@ -368,6 +414,8 @@ _con_curl_socketevent_cb (GIOChannel *ch, GIOCondition condition, gpointer user_
fdp->ev = 0;
}
+ _complete_queued (self);
+
return success ? G_SOURCE_CONTINUE : G_SOURCE_REMOVE;
}
@@ -419,10 +467,17 @@ easy_header_cb (char *buffer, size_t size, size_t nitems, void *userdata)
NMConnectivityCheckHandle *cb_data = userdata;
size_t len = size * nitems;
+ if (cb_data->completed_state != NM_CONNECTIVITY_UNKNOWN) {
+ /* already completed. */
+ return 0;
+ }
+
if ( len >= sizeof (HEADER_STATUS_ONLINE) - 1
&& !g_ascii_strncasecmp (buffer, HEADER_STATUS_ONLINE, sizeof (HEADER_STATUS_ONLINE) - 1)) {
- cb_data_invoke_callback (cb_data, NM_CONNECTIVITY_FULL,
- NULL, "status header found");
+ cb_data_queue_completed (cb_data,
+ NM_CONNECTIVITY_FULL,
+ "status header found",
+ NULL);
return 0;
}
@@ -434,24 +489,33 @@ easy_write_cb (void *buffer, size_t size, size_t nmemb, void *userdata)
{
NMConnectivityCheckHandle *cb_data = userdata;
size_t len = size * nmemb;
- const char *response = _check_handle_get_response (cb_data);;
+ const char *response;
+
+ if (cb_data->completed_state != NM_CONNECTIVITY_UNKNOWN) {
+ /* already completed. */
+ return 0;
+ }
if (!cb_data->concheck.recv_msg)
cb_data->concheck.recv_msg = g_string_sized_new (len + 10);
g_string_append_len (cb_data->concheck.recv_msg, buffer, len);
+ response = _check_handle_get_response (cb_data);;
if ( response
&& cb_data->concheck.recv_msg->len >= strlen (response)) {
/* We already have enough data -- check response */
if (g_str_has_prefix (cb_data->concheck.recv_msg->str, response)) {
- cb_data_invoke_callback (cb_data, NM_CONNECTIVITY_FULL, NULL,
- "expected response");
+ cb_data_queue_completed (cb_data,
+ NM_CONNECTIVITY_FULL,
+ "expected response",
+ NULL);
} else {
- cb_data_invoke_callback (cb_data, NM_CONNECTIVITY_PORTAL, NULL,
- "unexpected response");
+ cb_data_queue_completed (cb_data,
+ NM_CONNECTIVITY_PORTAL,
+ "unexpected response",
+ NULL);
}
-
return 0;
}
@@ -462,15 +526,11 @@ static gboolean
_timeout_cb (gpointer user_data)
{
NMConnectivityCheckHandle *cb_data = user_data;
- NMConnectivity *self;
nm_assert (NM_IS_CONNECTIVITY (cb_data->self));
+ nm_assert (c_list_contains (&NM_CONNECTIVITY_GET_PRIVATE (cb_data->self)->handles_lst_head, &cb_data->handles_lst));
- self = cb_data->self;
-
- nm_assert (c_list_contains (&NM_CONNECTIVITY_GET_PRIVATE (self)->handles_lst_head, &cb_data->handles_lst));
-
- cb_data_free (cb_data, NM_CONNECTIVITY_LIMITED, NULL, "timeout");
+ cb_data_complete (cb_data, NM_CONNECTIVITY_LIMITED, "timeout");
return G_SOURCE_REMOVE;
}
#endif
@@ -490,9 +550,9 @@ _idle_cb (gpointer user_data)
/* the invocation was with an invalid ifname. It is a fail. */
g_set_error (&error, NM_UTILS_ERROR, NM_UTILS_ERROR_INVALID_ARGUMENT,
"no interface specified for connectivity check");
- cb_data_free (cb_data, NM_CONNECTIVITY_ERROR, NULL, "missing interface");
+ cb_data_complete (cb_data, NM_CONNECTIVITY_ERROR, "missing interface");
} else
- cb_data_free (cb_data, NM_CONNECTIVITY_FAKE, NULL, "fake result");
+ cb_data_complete (cb_data, NM_CONNECTIVITY_FAKE, "fake result");
return G_SOURCE_REMOVE;
}
@@ -516,6 +576,7 @@ nm_connectivity_check_start (NMConnectivity *self,
c_list_link_tail (&priv->handles_lst_head, &cb_data->handles_lst);
cb_data->callback = callback;
cb_data->user_data = user_data;
+ cb_data->completed_state = NM_CONNECTIVITY_UNKNOWN;
if (iface)
cb_data->ifspec = g_strdup_printf ("if!%s", iface);
@@ -556,22 +617,13 @@ nm_connectivity_check_start (NMConnectivity *self,
void
nm_connectivity_check_cancel (NMConnectivityCheckHandle *cb_data)
{
- NMConnectivity *self;
- gs_free_error GError *error = NULL;
-
g_return_if_fail (cb_data);
+ g_return_if_fail (NM_IS_CONNECTIVITY (cb_data->self));
- self = cb_data->self;
-
- g_return_if_fail (NM_IS_CONNECTIVITY (self));
- g_return_if_fail (!c_list_is_empty (&cb_data->handles_lst));
- g_return_if_fail (cb_data->callback);
-
- nm_assert (c_list_contains (&NM_CONNECTIVITY_GET_PRIVATE (self)->handles_lst_head, &cb_data->handles_lst));
+ nm_assert ( c_list_contains (&NM_CONNECTIVITY_GET_PRIVATE (cb_data->self)->handles_lst_head, &cb_data->handles_lst)
+ || c_list_contains (&NM_CONNECTIVITY_GET_PRIVATE (cb_data->self)->completed_handles_lst_head, &cb_data->handles_lst));
- nm_utils_error_set_cancelled (&error, FALSE, "NMConnectivity");
-
- cb_data_free (cb_data, NM_CONNECTIVITY_ERROR, error, "cancelled");
+ cb_data_complete (cb_data, NM_CONNECTIVITY_CANCELLED, "cancelled");
}
/*****************************************************************************/
@@ -684,6 +736,7 @@ nm_connectivity_init (NMConnectivity *self)
NMConnectivityPrivate *priv = NM_CONNECTIVITY_GET_PRIVATE (self);
c_list_init (&priv->handles_lst_head);
+ c_list_init (&priv->completed_handles_lst_head);
priv->config = g_object_ref (nm_config_get ());
g_signal_connect (G_OBJECT (priv->config),
@@ -715,16 +768,13 @@ dispose (GObject *object)
NMConnectivity *self = NM_CONNECTIVITY (object);
NMConnectivityPrivate *priv = NM_CONNECTIVITY_GET_PRIVATE (self);
NMConnectivityCheckHandle *cb_data;
- GError *error = NULL;
-
-again:
- c_list_for_each_entry (cb_data, &priv->handles_lst_head, handles_lst) {
- if (!error)
- nm_utils_error_set_cancelled (&error, TRUE, "NMConnectivity");
- cb_data_free (cb_data, NM_CONNECTIVITY_ERROR, error, "shutting down");
- goto again;
- }
- g_clear_error (&error);
+
+ nm_assert (c_list_is_empty (&priv->completed_handles_lst_head));
+
+ while ((cb_data = c_list_first_entry (&priv->handles_lst_head,
+ NMConnectivityCheckHandle,
+ handles_lst)))
+ cb_data_complete (cb_data, NM_CONNECTIVITY_DISPOSING, "shutting down");
g_clear_pointer (&priv->uri, g_free);
g_clear_pointer (&priv->response, g_free);
diff --git a/src/nm-connectivity.h b/src/nm-connectivity.h
index df9295e02..178f27ad9 100644
--- a/src/nm-connectivity.h
+++ b/src/nm-connectivity.h
@@ -24,8 +24,10 @@
#include "nm-dbus-interface.h"
-#define NM_CONNECTIVITY_ERROR ((NMConnectivityState) -1)
-#define NM_CONNECTIVITY_FAKE ((NMConnectivityState) -2)
+#define NM_CONNECTIVITY_ERROR ((NMConnectivityState) -1)
+#define NM_CONNECTIVITY_FAKE ((NMConnectivityState) -2)
+#define NM_CONNECTIVITY_CANCELLED ((NMConnectivityState) -3)
+#define NM_CONNECTIVITY_DISPOSING ((NMConnectivityState) -4)
#define NM_TYPE_CONNECTIVITY (nm_connectivity_get_type ())
#define NM_CONNECTIVITY(obj) (G_TYPE_CHECK_INSTANCE_CAST ((obj), NM_TYPE_CONNECTIVITY, NMConnectivity))
@@ -53,7 +55,6 @@ typedef struct _NMConnectivityCheckHandle NMConnectivityCheckHandle;
typedef void (*NMConnectivityCheckCallback) (NMConnectivity *self,
NMConnectivityCheckHandle *handle,
NMConnectivityState state,
- GError *error,
gpointer user_data);
NMConnectivityCheckHandle *nm_connectivity_check_start (NMConnectivity *self,
--
2.17.1

View File

@ -8,8 +8,8 @@
%global glib2_version %(pkg-config --modversion glib-2.0 2>/dev/null || echo bad)
%global epoch_version 1
%global rpm_version 1.14.0
%global real_version 1.14.0
%global rpm_version 1.14.2
%global real_version 1.14.2
%global release_version 1
%global snapshot %{nil}
%global git_sha %{nil}
@ -36,6 +36,7 @@
###############################################################################
%bcond_with meson
%bcond_without adsl
%bcond_without bluetooth
%bcond_without wwan
@ -109,7 +110,7 @@ Source2: 00-server.conf
Source3: 20-connectivity-fedora.conf
Source4: 20-connectivity-redhat.conf
Patch1: 0001-connectivity-fix-crash-when-removing-easy-handle-fro.patch
#Patch1: 0001-some.patch
Requires(post): systemd
Requires(post): /usr/sbin/update-alternatives
@ -134,8 +135,12 @@ Conflicts: kde-plasma-networkmanagement < 1:0.9-0.49.20110527git.nm09
BuildRequires: gcc
BuildRequires: libtool
BuildRequires: pkgconfig
%if %{with meson}
BuildRequires: meson
%else
BuildRequires: automake
BuildRequires: autoconf
%endif
BuildRequires: intltool
BuildRequires: gettext-devel
@ -443,6 +448,99 @@ by nm-connection-editor and nm-applet in a non-graphical environment.
%build
%if %{with meson}
%meson \
-Ddhcpcanon=no \
-Ddhcpcd=no \
-Dconfig_dhcp_default=%{dhcp_default} \
%if %{with crypto_gnutls}
-Dcrypto=gnutls \
%else
-Dcrypto=nss \
%endif
%if %{with debug}
-Dmore_logging=true \
-Dmore_asserts=10000 \
%else
-Dmore_logging=false \
-Dmore_asserts=0 \
%endif
-Dld_gc=true \
-Dlibaudit=yes-disabled-by-default \
%if 0%{?with_modem_manager_1}
-Dmodem_manager=true \
%else
-Dmodem_manager=false \
%endif
%if %{with wifi}
-Dwifi=true \
%if 0%{?fedora}
-Dwext=true \
%else
-Dwext=false \
%endif
%else
-Dwifi=false \
%endif
%if %{with iwd}
-Diwd=true \
%else
-Diwd=false \
%endif
-Dvapi=true \
-Dintrospection=true \
%if %{with regen_docs}
-Ddocs=true \
%else
-Ddocs=false \
%endif
%if %{with team}
-Dteamdctl=true \
%else
-Dteamdctl=false \
%endif
%if %{with ovs}
-Dovs=true \
%else
-Dovs=false \
%endif
-Dselinux=true \
-Dpolkit=yes \
-Dpolkit_agent=true \
-Dmodify_system=true \
-Dconcheck=true \
%if 0%{?fedora}
-Dlibpsl=true \
%else
-Dlibpsl=false \
%endif
-Dsession_tracking=systemd \
-Dsuspend_resume=systemd \
-Dsystemdsystemunitdir=%{systemd_dir} \
-Dsystem_ca_path=/etc/pki/tls/cert.pem \
-Ddbus_conf_dir=%{dbus_sys_dir} \
-Dtests=yes \
-Dvalgrind=no \
-Difcfg_rh=true \
%if %{with ppp}
-Dpppd_plugin_dir=%{_libdir}/pppd/%{ppp_version} \
-Dppp=true \
%endif
-Ddist_version=%{version}-%{release} \
-Dconfig_plugins_default='ifcfg-rh' \
-Dconfig_dns_rc_manager_default=symlink \
-Dconfig_logging_backend_default=journal \
-Djson_validation=true \
%if %{with libnm_glib}
-Dlibnm_glib=true
%else
-Dlibnm_glib=false
%endif
%meson_build
%else
# autotools
%if %{with regen_docs}
gtkdocize
%endif
@ -543,7 +641,7 @@ intltoolize --automake --copy --force
--enable-ppp=yes \
%endif
--with-dist-version=%{version}-%{release} \
--with-config-plugins-default='ifcfg-rh,ibft' \
--with-config-plugins-default='ifcfg-rh' \
--with-config-dns-rc-manager-default=symlink \
--with-config-logging-backend-default=journal \
--enable-json-validation \
@ -555,10 +653,14 @@ intltoolize --automake --copy --force
make %{?_smp_mflags}
%endif # end autotools
%install
# install NM
%if %{with meson}
%meson_install
%else
make install DESTDIR=%{buildroot}
%endif
cp %{SOURCE1} %{buildroot}%{_sysconfdir}/%{name}/
@ -594,11 +696,20 @@ touch %{buildroot}%{_sbindir}/ifup %{buildroot}%{_sbindir}/ifdown
%check
%if %{with meson}
%if %{with test}
%meson_test
%else
%ninja_test -C %{_vpath_builddir} || :
%endif
%else
# autotools
%if %{with test}
make -k %{?_smp_mflags} check
%else
make -k %{?_smp_mflags} check || :
%endif
%endif # end autotools
%pre
@ -677,6 +788,7 @@ fi
%{_libexecdir}/nm-dhcp-helper
%{_libexecdir}/nm-dispatcher
%{_libexecdir}/nm-iface-helper
%{_libexecdir}/nm-initrd-generator
%dir %{_libdir}/%{name}
%dir %{nmplugindir}
%{nmplugindir}/libnm-settings-plugin*.so
@ -769,7 +881,6 @@ fi
%if %{with libnm_glib}
%files glib-devel
%doc docs/api/html/*
%dir %{_includedir}/libnm-glib
%dir %{_includedir}/%{name}
%{_includedir}/libnm-glib/*.h
@ -805,7 +916,6 @@ fi
%files libnm-devel
%doc docs/api/html/*
%dir %{_includedir}/libnm
%{_includedir}/libnm/*.h
%{_libdir}/pkgconfig/libnm.pc
@ -859,6 +969,9 @@ fi
%changelog
* Fri Oct 19 2018 Lubomir Rintel <lkundrak@v3.sk> - 1:1.14.2-1
- Update to 1.14.2 release
* Tue Sep 18 2018 Thomas Haller <thaller@redhat.com> - 1:1.14.0-1
- Update to 1.14.0 release

View File

@ -1 +1 @@
SHA512 (NetworkManager-1.14.0.tar.xz) = cb0add2585c586ad36b4fb986a654de30047dc77f038927437156227f42622ac1045c58b76573f4e1acef22aa9f695da417f392a50fde70f4a086504701cecf1
SHA512 (NetworkManager-1.14.2.tar.xz) = f2c45138daf8124704d4f8ee3966da8a2b6c9a18bc51332d19c36d6d5bbd795c6f85f182c7816604fa5e8be7e6ee1daa62b26a868b731e7bc4d4b72e543d3646