开源软件构建与测试

We truncated the diff of some files because they were too big. If you want to see the full diff for every file, click here.

Changes of Revision 2

_service:extract_file:ModemManager.spec Deleted

@@ -1,172 +0,0 @@
-%global glib2_version %(pkg-config --modversion glib-2.0 2>/dev/null || echo bad)
-%global qmi_version %(pkg-config --modversion qmi-glib 2>/dev/null || echo bad)
-%global mbim_version %(pkg-config --modversion mbim-glib 2>/dev/null || echo bad)
-
-
-Name:           ModemManager
-Version:        1.14.8
-Release:        1
-Summary:        Mobile broadband modem manager
-License:        GPLv2+
-URL:            https://www.freedesktop.org/wiki/Software/ModemManager/
-Source:         https://www.freedesktop.org/software/ModemManager/%{name}-%{version}.tar.xz
-
-BuildRequires:  automake autoconf dbus gtk-doc libtool vala-tools vala-devel
-BuildRequires:  gettext-devel glib2-devel gobject-introspection-devel libgudev-devel
-BuildRequires:  libqmi-devel libmbim-devel systemd-devel
-BuildRequires:  autoconf-archive 
-
-Requires:       libmbim-utils libqmi-utils glib2 >= %{glib2_version}
-Requires:       libmbim >= %{mbim_version} libqmi >= %{qmi_version}
-Requires:       %{name}-glib = %{version}-%{release}
-%{?systemd_requires}
-
-%description
-ModemManager is a DBus-activated daemon which controls mobile broadband
-(2G/3G/4G) devices and connections. Whether built-in devices, USB dongles,
-bluetooth-paired telephones, or professional RS232/USB devices with
-external power supplies, ModemManager is able to prepare and configure
-the modems and setup connections with them.
-
-%package        devel
-Summary:        Development package for ModemManager
-Requires:       %{name} = %{version}-%{release} pkgconfig
-
-%description    devel
-%{name}-devel contains header files and libraries for %{name}.
-
-%package        glib
-Summary:        Libraries for applications using glib to use ModemManager
-Requires:       glib2 >= %{glib2_version}
-
-%description    glib
-Libraries for applications using glib to use ModemManager.
-
-%package        glib-devel
-Summary:        Development package for applications using glib to use ModemManager
-Requires:       pkgconfig
-Requires:       %{name} = %{version}-%{release}
-Requires:       %{name}-devel = %{version}-%{release}
-Requires:       %{name}-glib = %{version}-%{release}
-Requires:       glib2-devel >= %{glib2_version}
-
-%description    glib-devel
-Development package for applications using glib to use ModemManager.
-
-%package        vala
-Summary:        Vala bindings for ModemManager
-Requires:       vala
-Requires:       %{name}-glib = %{version}-%{release}
-
-%description vala
-Vala bindings for ModemManager
-
-%package_help
-
-%prep
-%autosetup -n %{name}-%{version}
-
-%build
-autoreconf -if
-%configure \
-        --enable-gtk-doc  --enable-more-warnings=no  --disable-rpath \
-        --disable-silent-rules  --with-udev-base-dir=%{_prefix}/lib/udev \
-        --with-qmi=yes  --with-mbim=yes  --with-polkit=no  --with-dist-version=%{version}-%{release} \
-        --with-systemd-journal  --with-systemd-suspend-resume
-%make_build
-
-%check
-make check
-
-%install
-%make_install
-%delete_la
-
-%post
-%systemd_post ModemManager.service
-
-%preun
-%systemd_preun ModemManager.service
-
-%ldconfig_scriptlets glib
-
-%files
-%defattr(-,root,root)
-%doc AUTHORS
-%license COPYING COPYING.LIB  
-%{_bindir}/mmcli
-%{_sbindir}/ModemManager
-%dir %{_libdir}/%{name}
-%{_libdir}/%{name}/*.so
-%{_datadir}/locale/*
-%{_udevrulesdir}/*
-%{_unitdir}/ModemManager.service
-%{_datadir}/dbus-1/system-services/org.freedesktop.ModemManager1.service
-%{_datadir}/bash-completion
-%{_sysconfdir}/dbus-1/system.d/org.freedesktop.ModemManager1.conf
-%{_datadir}/ModemManager
-
-%files devel
-%defattr(-,root,root)
-%{_libdir}/pkgconfig/%{name}.pc
-%{_includedir}/ModemManager/
-
-%files glib
-%defattr(-,root,root)
-%{_libdir}/libmm-glib.so.*
-%{_libdir}/girepository-1.0/*.typelib
-
-%files glib-devel
-%defattr(-,root,root)
-%{_libdir}/libmm-glib.so
-%{_libdir}/pkgconfig/mm-glib.pc
-%{_includedir}/libmm-glib/*.h
-%{_datadir}/gtk-doc/html/libmm-glib/*
-%{_datadir}/gir-1.0/*.gir
-
-%files vala
-%defattr(-,root,root)
-%{_datadir}/vala/vapi/libmm-glib.*
-
-%files help
-%defattr(-,root,root)
-%doc README TODO NEWS ChangeLog
-%{_mandir}/man8/*
-%{_mandir}/man1/*
-%{_datadir}/icons/hicolor/22x22/apps/*.png
-%{_datadir}/gtk-doc/html/*/*
-%{_datadir}/dbus-1/interfaces/*.xml
-
-%changelog
-* Wed Jan 27 2021 xihaochen <xihaochen@huawei.com> - 1.14.8-1
-- Type:requirements
-- ID:NA
-- SUG:NA
-- DESC: update ModemManager to 1.14.8
-
-* Thu Dec 03 2020 liulong <liulong20@huawei.com> - 1.14.0-9
-- Type:bugfix
-- Id:NA
-- SUG:NA
-- DESC:fix service startup failure.
-
-* Fri Jul 17 2020 cuibaobao <cuibaobao1@huawei.com> - 1.14.0-8
-- Type:update
-- Id:NA
-- SUG:NA
-- DESC:update to 1.14.0
-
-* Sat Oct 19 2019 openEuler Buildteam <buildteam@openeuler.org> - 1.8.0-7
-- Type:bugfix
-- Id:NA
-- SUG:NA
-- DESC:change the directory of the license file
-
-* Thu Sep 05 2019 openEuler Buildteam <buildteam@openeuler.org> - 1.8.0-6
-- Type:enhance
-- ID:NA
-- SUG:NA
-- DESC:new rule
-
-* Thu Aug 22 2019 openEuler Buildteam <buildteam@openeuler.org> - 1.8.0-5
-- Package init

_service:tar_scm:ModemManager.spec Added

@@ -0,0 +1,190 @@
+%global glib2_version %(pkg-config --modversion glib-2.0 2>/dev/null || echo bad)
+%global qmi_version %(pkg-config --modversion qmi-glib 2>/dev/null || echo bad)
+%global mbim_version %(pkg-config --modversion mbim-glib 2>/dev/null || echo bad)
+
+
+Name:           ModemManager
+Version:        1.18.12
+Release:        1
+Summary:        Mobile broadband modem manager
+License:        GPLv2+
+URL:            https://www.freedesktop.org/wiki/Software/ModemManager/
+Source:         https://www.freedesktop.org/software/ModemManager/%{name}-%{version}.tar.xz
+
+Patch2:         backport-core-port-GRegex-GMatchInfo-to-use-autoptr.patch
+Patch5:         backport-modem-helpers-rework-CGCONTRDP-response-parser.patch
+Patch6:         backport-modem-helpers-cinterion-rework-CNMI-test-response-parser.patch
+Patch7:         backport-modem-helpers-ublox-rework-UIPADDR-response-parser.patch
+Patch8:         backport-modem-helpers-rework-the-CGEV-indication-parser.patch
+
+BuildRequires:  automake autoconf dbus gtk-doc libtool vala-tools vala-devel
+BuildRequires:  gettext-devel glib2-devel gobject-introspection-devel libgudev-devel
+BuildRequires:  libqmi-devel >= 1.30.8 libmbim-devel >= 1.26.0 systemd-devel
+BuildRequires:  autoconf-archive python3-gobject python3-dbus
+
+Requires:       libmbim-utils libqmi-utils glib2 >= %{glib2_version}
+Requires:       libmbim >= %{mbim_version} libqmi >= %{qmi_version}
+Requires:       %{name}-glib = %{version}-%{release}
+%{?systemd_requires}
+
+%description
+ModemManager is a DBus-activated daemon which controls mobile broadband
+(2G/3G/4G) devices and connections. Whether built-in devices, USB dongles,
+bluetooth-paired telephones, or professional RS232/USB devices with
+external power supplies, ModemManager is able to prepare and configure
+the modems and setup connections with them.
+
+%package        devel
+Summary:        Development package for ModemManager
+Requires:       %{name} = %{version}-%{release} pkgconfig
+
+%description    devel
+%{name}-devel contains header files and libraries for %{name}.
+
+%package        glib
+Summary:        Libraries for applications using glib to use ModemManager
+Requires:       glib2 >= %{glib2_version}
+
+%description    glib
+Libraries for applications using glib to use ModemManager.
+
+%package        glib-devel
+Summary:        Development package for applications using glib to use ModemManager
+Requires:       pkgconfig
+Requires:       %{name} = %{version}-%{release}
+Requires:       %{name}-devel = %{version}-%{release}
+Requires:       %{name}-glib = %{version}-%{release}
+Requires:       glib2-devel >= %{glib2_version}
+
+%description    glib-devel
+Development package for applications using glib to use ModemManager.
+
+%package        vala
+Summary:        Vala bindings for ModemManager
+Requires:       vala
+Requires:       %{name}-glib = %{version}-%{release}
+
+%description vala
+Vala bindings for ModemManager
+
+%package_help
+
+%prep
+%autosetup -n %{name}-%{version} -p1
+
+%build
+autoreconf -if
+%configure \
+        --enable-gtk-doc  --enable-more-warnings=no  --disable-rpath \
+        --disable-silent-rules  --with-udev-base-dir=%{_prefix}/lib/udev \
+        --with-qmi=yes  --with-mbim=yes  --with-polkit=no  --with-dist-version=%{version}-%{release} \
+        --with-systemd-journal  --with-systemd-suspend-resume
+%make_build
+
+%check
+make check
+
+%install
+%make_install
+%delete_la
+
+%post
+%systemd_post ModemManager.service
+
+%preun
+%systemd_preun ModemManager.service
+
+%ldconfig_scriptlets glib
+
+%files
+%defattr(-,root,root)
+%doc AUTHORS
+%license COPYING COPYING.LIB  
+%{_bindir}/mmcli
+%{_sbindir}/ModemManager
+%dir %{_libdir}/%{name}
+%{_libdir}/%{name}/*.so
+%{_datadir}/locale/*
+%{_udevrulesdir}/*
+%{_unitdir}/ModemManager.service
+%{_datadir}/dbus-1/system-services/org.freedesktop.ModemManager1.service
+%{_datadir}/bash-completion
+%{_sysconfdir}/dbus-1/system.d/org.freedesktop.ModemManager1.conf
+%{_datadir}/ModemManager
+
+%files devel
+%defattr(-,root,root)
+%{_libdir}/pkgconfig/%{name}.pc
+%{_includedir}/ModemManager/
+
+%files glib
+%defattr(-,root,root)
+%{_libdir}/libmm-glib.so.*
+%{_libdir}/girepository-1.0/*.typelib
+
+%files glib-devel
+%defattr(-,root,root)
+%{_libdir}/libmm-glib.so
+%{_libdir}/pkgconfig/mm-glib.pc
+%{_includedir}/libmm-glib/*.h
+%{_datadir}/gtk-doc/html/libmm-glib/*
+%{_datadir}/gir-1.0/*.gir
+
+%files vala
+%defattr(-,root,root)
+%{_datadir}/vala/vapi/libmm-glib.*
+
+%files help
+%defattr(-,root,root)
+%doc README TODO NEWS ChangeLog
+%{_mandir}/man8/*
+%{_mandir}/man1/*
+%{_datadir}/icons/hicolor/22x22/apps/*.png
+%{_datadir}/gtk-doc/html/*/*
+%{_datadir}/dbus-1/interfaces/*.xml
+
+%changelog
+* Fri Nov 18 2022 gaihuiying <eaglegai@163.com> - 1.18.12-1
+- Type:requirements
+- ID:NA
+- SUG:NA
+- DESC: update ModemManager to 1.18.12
+
+* Sat Nov 12 2022 gaihuiying <eaglegai@163.com> - 1.14.8-2
+- Type:bugfix
+- ID:NA
+- SUG:NA
+- DESC: fix test failure because of glib2 switching to pcre2
+
+* Wed Jan 27 2021 xihaochen <xihaochen@huawei.com> - 1.14.8-1
+- Type:requirements
+- ID:NA
+- SUG:NA
+- DESC: update ModemManager to 1.14.8
+
+* Thu Dec 03 2020 liulong <liulong20@huawei.com> - 1.14.0-9
+- Type:bugfix
+- Id:NA
+- SUG:NA
+- DESC:fix service startup failure.
+
+* Fri Jul 17 2020 cuibaobao <cuibaobao1@huawei.com> - 1.14.0-8
+- Type:update
+- Id:NA
+- SUG:NA
+- DESC:update to 1.14.0
+
+* Sat Oct 19 2019 openEuler Buildteam <buildteam@openeuler.org> - 1.8.0-7
+- Type:bugfix
+- Id:NA
+- SUG:NA
+- DESC:change the directory of the license file
+
+* Thu Sep 05 2019 openEuler Buildteam <buildteam@openeuler.org> - 1.8.0-6
+- Type:enhance
+- ID:NA
+- SUG:NA
+- DESC:new rule
+
+* Thu Aug 22 2019 openEuler Buildteam <buildteam@openeuler.org> - 1.8.0-5
+- Package init

_service:tar_scm:backport-core-port-GRegex-GMatchInfo-to-use-autoptr.patch Added

@@ -0,0 +1,4337 @@
+From 74fc5baca229922b244ad66d5819d6d765e2d9da Mon Sep 17 00:00:00 2001
+From: Aleksander Morgado <aleksandermj@chromium.org>
+Date: Wed, 24 Aug 2022 12:31:47 +0000
+Subject: PATCH core: port GRegex/GMatchInfo to use autoptr()
+
+The behavior of GRegex changed in 2.73.2 once it was ported from pcre1
+to pcre2. In some cases it was made more strict, which is fine, in
+other cases it exposed some change in how it behaves on certain
+matches that is not extremely clear whether it's ok or not.
+
+See https://gitlab.gnome.org/GNOME/glib/-/issues/2729
+See https://gitlab.freedesktop.org/mobile-broadband/ModemManager/-/issues/601
+See https://gitlab.freedesktop.org/mobile-broadband/ModemManager/-/issues/621
+
+Either way, one thing that was assumed was that initializing all
+GRegex/GMatchInfo variables to NULL and making sure they're NULL
+before they're initialized by glib (especially the GMatchInfo) was a
+good and safer approach.
+
+So, whenever possible, g_autoptr() is used to cleanup the allocated
+GMatchInfo/GRegex variables, and otherwise, g_clear_pointer() is used
+to ensure that no free/unref is attempted unless the given variable is
+not NULL, and also so that the variable is reseted to NULL after being
+disposed.
+---
+ libmm-glib/mm-location-gps-nmea.c             |  10 +-
+ libmm-glib/mm-location-gps-raw.c              |   4 +-
+ plugins/anydata/mm-broadband-modem-anydata.c  |  82 ++-
+ .../cinterion/mm-modem-helpers-cinterion.c    |  62 +-
+ .../tests/test-modem-helpers-cinterion.c      |  42 +-
+ plugins/huawei/mm-broadband-modem-huawei.c    |  19 +-
+ plugins/huawei/mm-modem-helpers-huawei.c      | 329 +++++------
+ plugins/icera/mm-broadband-modem-icera.c      |  32 +-
+ plugins/mbm/mm-modem-helpers-mbm.c            |  37 +-
+ plugins/mtk/mm-broadband-modem-mtk.c          | 104 ++--
+ plugins/novatel/mm-broadband-modem-novatel.c  |  45 +-
+ plugins/option/mm-broadband-modem-option.c    |  42 +-
+ plugins/quectel/mm-shared-quectel.c           |   3 +-
+ plugins/sierra/mm-broadband-modem-sierra.c    |  41 +-
+ plugins/sierra/mm-common-sierra.c             |   4 +-
+ plugins/sierra/mm-modem-helpers-sierra.c      |  16 +-
+ .../tests/test-modem-helpers-simtech.c        |  75 +--
+ plugins/telit/mm-broadband-modem-telit.c      |   9 +-
+ plugins/telit/mm-common-telit.c               |  16 +-
+ plugins/telit/mm-modem-helpers-telit.c        |  13 +-
+ plugins/ublox/mm-broadband-modem-ublox.c      |  11 +-
+ plugins/ublox/mm-modem-helpers-ublox.c        | 152 ++---
+ plugins/via/mm-broadband-modem-via.c          |  22 +-
+ plugins/wavecom/mm-broadband-modem-wavecom.c  |   7 +-
+ plugins/x22x/mm-broadband-modem-x22x.c        |  17 +-
+ plugins/xmm/mm-modem-helpers-xmm.c            |  51 +-
+ plugins/zte/mm-broadband-modem-zte.c          |  28 +-
+ src/mm-broadband-modem.c                      | 187 ++----
+ src/mm-modem-helpers.c                        | 538 +++++++-----------
+ src/mm-port-serial-at.c                       |  11 +-
+ src/mm-port-serial-gps.c                      |  14 +-
+ src/mm-serial-parsers.c                       |  21 +-
+ src/tests/test-modem-helpers.c                |  72 +--
+ 33 files changed, 849 insertions(+), 1267 deletions(-)
+
+diff --git a/libmm-glib/mm-location-gps-nmea.c b/libmm-glib/mm-location-gps-nmea.c
+index d23bc1462..b2ef69511 100644
+--- a/libmm-glib/mm-location-gps-nmea.c
++++ b/libmm-glib/mm-location-gps-nmea.c
+@@ -54,9 +54,7 @@ static gboolean
+ check_append_or_replace (MMLocationGpsNmea *self,
+                          const gchar *trace)
+ {
+-    /* By default, replace */
+-    gboolean append_or_replace = FALSE;
+-    GMatchInfo *match_info = NULL;
++    g_autoptr(GMatchInfo) match_info = NULL;
+ 
+     if (G_UNLIKELY (!self->priv->sequence_regex))
+         self->priv->sequence_regex = g_regex_new ("\\$..(?:ALM|GSV|RTE|SFI),(\\d),(\\d).*",
+@@ -69,11 +67,11 @@ check_append_or_replace (MMLocationGpsNmea *self,
+ 
+         /* If we don't have the first element of a sequence, append */
+         if (mm_get_uint_from_match_info (match_info, 2, &index) && index != 1)
+-            append_or_replace = TRUE;
++            return TRUE;
+     }
+-    g_match_info_free (match_info);
+ 
+-    return append_or_replace;
++    /* By default, replace */
++    return FALSE;
+ }
+ 
+ static gboolean
+diff --git a/libmm-glib/mm-location-gps-raw.c b/libmm-glib/mm-location-gps-raw.c
+index fddc5629b..d776d1ca6 100644
+--- a/libmm-glib/mm-location-gps-raw.c
++++ b/libmm-glib/mm-location-gps-raw.c
+@@ -190,7 +190,7 @@ gboolean
+ mm_location_gps_raw_add_trace (MMLocationGpsRaw *self,
+                                const gchar *trace)
+ {
+-    GMatchInfo *match_info = NULL;
++    g_autoptr(GMatchInfo) match_info = NULL;
+ 
+     /* Current implementation works only with $GPGGA and $GNGGA traces */
+     do {
+@@ -268,8 +268,6 @@ mm_location_gps_raw_add_trace (MMLocationGpsRaw *self,
+         mm_get_double_from_match_info (match_info, 9, &self->priv->altitude);
+     }
+ 
+-    g_match_info_free (match_info);
+-
+     return TRUE;
+ }
+ 
+diff --git a/plugins/anydata/mm-broadband-modem-anydata.c b/plugins/anydata/mm-broadband-modem-anydata.c
+index 95f8cbd27..36d72e56c 100644
+--- a/plugins/anydata/mm-broadband-modem-anydata.c
++++ b/plugins/anydata/mm-broadband-modem-anydata.c
+@@ -72,10 +72,10 @@ hstate_ready (MMIfaceModemCdma *self,
+               GTask *task)
+ {
+     DetailedRegistrationStateResults *results;
+-    GError *error = NULL;
+-    const gchar *response;
+-    GRegex *r;
+-    GMatchInfo *match_info;
++    GError                           *error = NULL;
++    const gchar                      *response;
++    g_autoptr(GRegex)                 r = NULL;
++    g_autoptr(GMatchInfo)             match_info = NULL;
+ 
+     results = g_task_get_task_data (task);
+ 
+@@ -129,9 +129,6 @@ hstate_ready (MMIfaceModemCdma *self,
+         }
+     }
+ 
+-    g_match_info_free (match_info);
+-    g_regex_unref (r);
+-
+     g_task_return_pointer (task, g_memdup (results, sizeof (*results)), g_free);
+     g_object_unref (task);
+ }
+@@ -142,10 +139,10 @@ state_ready (MMIfaceModemCdma *self,
+              GTask *task)
+ {
+     DetailedRegistrationStateResults *results;
+-    GError *error = NULL;
+-    const gchar *response;
+-    GRegex *r;
+-    GMatchInfo *match_info;
++    GError                           *error = NULL;
++    const gchar                      *response;
++    g_autoptr(GRegex)                 r = NULL;
++    g_autoptr(GMatchInfo)             match_info = NULL;
+ 
+     response = mm_base_modem_at_command_finish (MM_BASE_MODEM (self), res, &error);
+     if (error) {
+@@ -194,9 +191,6 @@ state_ready (MMIfaceModemCdma *self,
+         }
+     }
+ 
+-    g_match_info_free (match_info);
+-    g_regex_unref (r);
+-
+     /* Try for EVDO state too */
+     mm_base_modem_at_command (MM_BASE_MODEM (self),
+                               "*HSTATE?",
+@@ -261,9 +255,14 @@ reset (MMIfaceModem *self,
+ static void
+ setup_ports (MMBroadbandModem *self)
+ {
+-    MMPortSerialAt *ports2;
+-    GRegex *regex;
+-    guint i;
++    MMPortSerialAt    *ports2;
++    g_autoptr(GRegex)  active_regex = NULL;
++    g_autoptr(GRegex)  inactive_regex = NULL;
++    g_autoptr(GRegex)  dormant_regex = NULL;
++    g_autoptr(GRegex)  offline_regex = NULL;
++    g_autoptr(GRegex)  regreq_regex = NULL;
++    g_autoptr(GRegex)  authreq_regex = NULL;
++    guint              i;
+ 
+     /* Call parent's setup ports first always */
+     MM_BROADBAND_MODEM_CLASS (mm_broadband_modem_anydata_parent_class)->setup_ports (self);
+@@ -271,48 +270,37 @@ setup_ports (MMBroadbandModem *self)
+     ports0 = mm_base_modem_peek_port_primary (MM_BASE_MODEM (self));
+     ports1 = mm_base_modem_peek_port_secondary (MM_BASE_MODEM (self));
+ 
++    /* Data call has connected */
++    active_regex = g_regex_new ("\\r\\n\\*ACTIVE:(.*)\\r\\n", G_REGEX_RAW | G_REGEX_OPTIMIZE, 0, NULL);
++    /* Data call disconnected */
++    inactive_regex = g_regex_new ("\\r\\n\\*INACTIVE:(.*)\\r\\n", G_REGEX_RAW | G_REGEX_OPTIMIZE, 0, NULL);
++    /* Modem is now dormant */
++    dormant_regex = g_regex_new ("\\r\\n\\*DORMANT:(.*)\\r\\n", G_REGEX_RAW | G_REGEX_OPTIMIZE, 0, NULL);
++    /* Network acquisition fail */
++    offline_regex = g_regex_new ("\\r\\n\\*OFFLINE:(.*)\\r\\n", G_REGEX_RAW | G_REGEX_OPTIMIZE, 0, NULL);
++    /* Registration fail */
++    regreq_regex = g_regex_new ("\\r\\n\\*REGREQ:(.*)\\r\\n", G_REGEX_RAW | G_REGEX_OPTIMIZE, 0, NULL);
++    /* Authentication fail */

_service:tar_scm:backport-modem-helpers-cinterion-rework-CNMI-test-response-parser.patch Added

@@ -0,0 +1,66 @@
+From e6c40349b81ac2915ce90b7e399eb33719a1fd24 Mon Sep 17 00:00:00 2001
+From: Aleksander Morgado <aleksandermj@chromium.org>
+Date: Wed, 24 Aug 2022 16:42:45 +0000
+Subject: PATCH modem-helpers-cinterion: rework CNMI test response parser
+
+We setup all output variables with g_autoptr() and then use
+g_steal_pointer() to return the needed ones.
+---
+ .../cinterion/mm-modem-helpers-cinterion.c    | 25 ++++++++-----------
+ 1 file changed, 10 insertions(+), 15 deletions(-)
+
+diff --git a/plugins/cinterion/mm-modem-helpers-cinterion.c b/plugins/cinterion/mm-modem-helpers-cinterion.c
+index 0bf3a0782..9219bff4d 100644
+--- a/plugins/cinterion/mm-modem-helpers-cinterion.c
++++ b/plugins/cinterion/mm-modem-helpers-cinterion.c
+@@ -556,12 +556,12 @@ mm_cinterion_parse_cnmi_test (const gchar *response,
+ {
+     g_autoptr(GRegex)      r = NULL;
+     g_autoptr(GMatchInfo)  match_info = NULL;
++    g_autoptr(GArray)      tmp_supported_mode = NULL;
++    g_autoptr(GArray)      tmp_supported_mt = NULL;
++    g_autoptr(GArray)      tmp_supported_bm = NULL;
++    g_autoptr(GArray)      tmp_supported_ds = NULL;
++    g_autoptr(GArray)      tmp_supported_bfr = NULL;
+     GError                *inner_error = NULL;
+-    GArray                *tmp_supported_mode = NULL;
+-    GArray                *tmp_supported_mt = NULL;
+-    GArray                *tmp_supported_bm = NULL;
+-    GArray                *tmp_supported_ds = NULL;
+-    GArray                *tmp_supported_bfr = NULL;
+ 
+     if (!response) {
+         g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Missing response");
+@@ -619,25 +619,20 @@ mm_cinterion_parse_cnmi_test (const gchar *response,
+ 
+ out:
+     if (inner_error) {
+-        g_clear_pointer (&tmp_supported_mode, g_array_unref);
+-        g_clear_pointer (&tmp_supported_mt,   g_array_unref);
+-        g_clear_pointer (&tmp_supported_bm,   g_array_unref);
+-        g_clear_pointer (&tmp_supported_ds,   g_array_unref);
+-        g_clear_pointer (&tmp_supported_bfr,  g_array_unref);
+         g_propagate_error (error, inner_error);
+         return FALSE;
+     }
+ 
+     if (supported_mode)
+-        *supported_mode = tmp_supported_mode;
++        *supported_mode = g_steal_pointer (&tmp_supported_mode);
+     if (supported_mt)
+-        *supported_mt = tmp_supported_mt;
++        *supported_mt = g_steal_pointer (&tmp_supported_mt);
+     if (supported_bm)
+-        *supported_bm = tmp_supported_bm;
++        *supported_bm = g_steal_pointer (&tmp_supported_bm);
+     if (supported_ds)
+-        *supported_ds = tmp_supported_ds;
++        *supported_ds = g_steal_pointer (&tmp_supported_ds);
+     if (supported_bfr)
+-        *supported_bfr = tmp_supported_bfr;
++        *supported_bfr = g_steal_pointer (&tmp_supported_bfr);
+ 
+     return TRUE;
+ }
+-- 
+GitLab

_service:tar_scm:backport-modem-helpers-rework-CGCONTRDP-response-parser.patch Added

@@ -0,0 +1,156 @@
+From 845667c7b3d36f0e776d5a0e582ad62d48cdeb02 Mon Sep 17 00:00:00 2001
+From: Aleksander Morgado <aleksandermj@chromium.org>
+Date: Wed, 24 Aug 2022 16:41:36 +0000
+Subject: PATCH modem-helpers: rework +CGCONTRDP response parser
+
+We setup all output variables with g_autofree and then use
+g_steal_pointer() to return the needed ones.
+---
+ src/mm-modem-helpers.c | 87 +++++++++++++++---------------------------
+ 1 file changed, 31 insertions(+), 56 deletions(-)
+
+diff --git a/src/mm-modem-helpers.c b/src/mm-modem-helpers.c
+index 58207bb15..9c3b0ccdc 100644
+--- a/src/mm-modem-helpers.c
++++ b/src/mm-modem-helpers.c
+@@ -2335,13 +2335,13 @@ mm_3gpp_parse_cgcontrdp_response (const gchar  *response,
+     GError                *inner_error = NULL;
+     guint                  cid = 0;
+     guint                  bearer_id = 0;
+-    gchar                 *apn = NULL;
+-    gchar                 *local_address_and_subnet = NULL;
+-    gchar                 *local_address = NULL;
+-    gchar                 *subnet = NULL;
+-    gchar                 *gateway_address = NULL;
+-    gchar                 *dns_primary_address = NULL;
+-    gchar                 *dns_secondary_address = NULL;
++    g_autofree gchar      *apn = NULL;
++    g_autofree gchar      *local_address_and_subnet = NULL;
++    g_autofree gchar      *local_address = NULL;
++    g_autofree gchar      *subnet = NULL;
++    g_autofree gchar      *gateway_address = NULL;
++    g_autofree gchar      *dns_primary_address = NULL;
++    g_autofree gchar      *dns_secondary_address = NULL;
+     guint                  field_format_extra_index = 0;
+ 
+     /* Response may be e.g.:
+@@ -2371,28 +2371,28 @@ mm_3gpp_parse_cgcontrdp_response (const gchar  *response,
+     g_assert (r != NULL);
+ 
+     g_regex_match_full (r, response, strlen (response), 0, 0, &match_info, &inner_error);
+-    if (inner_error)
+-        goto out;
++    if (inner_error) {
++        g_propagate_error (error, inner_error);
++        return FALSE;
++    }
+ 
+     if (!g_match_info_matches (match_info)) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_INVALID_ARGS, "Couldn't match +CGCONTRDP response");
+-        goto out;
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_INVALID_ARGS, "Couldn't match +CGCONTRDP response");
++        return FALSE;
+     }
+ 
+     if (out_cid && !mm_get_uint_from_match_info (match_info, 1, &cid)) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing cid");
+-        goto out;
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing cid");
++        return FALSE;
+     }
+ 
+     if (out_bearer_id && !mm_get_uint_from_match_info (match_info, 2, &bearer_id)) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing bearer id");
+-        goto out;
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing bearer id");
++        return FALSE;
+     }
+ 
+     /* Remaining strings are optional or empty allowed */
+-
+-    if (out_apn)
+-        apn = mm_get_string_unquoted_from_match_info (match_info, 3);
++    apn = mm_get_string_unquoted_from_match_info (match_info, 3);
+ 
+     /*
+      * The +CGCONTRDP=cid response format before version TS 27.007 v9.4.0 had
+@@ -2400,64 +2400,39 @@ mm_3gpp_parse_cgcontrdp_response (const gchar  *response,
+      */
+     local_address_and_subnet = mm_get_string_unquoted_from_match_info (match_info, 4);
+     if (local_address_and_subnet && !split_local_address_and_subnet (local_address_and_subnet, &local_address, &subnet)) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing local address and subnet");
+-        goto out;
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing local address and subnet");
++        return FALSE;
+     }
++
+     /* If we don't have a subnet in field 4, we're using the old format with subnet in an extra field */
+     if (!subnet) {
+-        if (out_subnet)
+-            subnet = mm_get_string_unquoted_from_match_info (match_info, 5);
++        subnet = mm_get_string_unquoted_from_match_info (match_info, 5);
+         field_format_extra_index = 1;
+     }
+ 
+-    if (out_gateway_address)
+-        gateway_address = mm_get_string_unquoted_from_match_info (match_info, 5 + field_format_extra_index);
+-
+-    if (out_dns_primary_address)
+-        dns_primary_address = mm_get_string_unquoted_from_match_info (match_info, 6 + field_format_extra_index);
+-
+-    if (out_dns_secondary_address)
+-        dns_secondary_address = mm_get_string_unquoted_from_match_info (match_info, 7 + field_format_extra_index);
+-
+-out:
+-    g_free (local_address_and_subnet);
+-
+-    if (inner_error) {
+-        g_free (apn);
+-        g_free (local_address);
+-        g_free (subnet);
+-        g_free (gateway_address);
+-        g_free (dns_primary_address);
+-        g_free (dns_secondary_address);
+-        g_propagate_error (error, inner_error);
+-        return FALSE;
+-    }
++    gateway_address = mm_get_string_unquoted_from_match_info (match_info, 5 + field_format_extra_index);
++    dns_primary_address = mm_get_string_unquoted_from_match_info (match_info, 6 + field_format_extra_index);
++    dns_secondary_address = mm_get_string_unquoted_from_match_info (match_info, 7 + field_format_extra_index);
+ 
+     if (out_cid)
+         *out_cid = cid;
+     if (out_bearer_id)
+         *out_bearer_id = bearer_id;
+     if (out_apn)
+-        *out_apn = apn;
+-
++        *out_apn = g_steal_pointer (&apn);
+     /* Local address and subnet may always be retrieved, even if not requested
+      * by the caller, as we need them to know which +CGCONTRDP=cid response is
+      * being parsed. So make sure we free them if not needed. */
+     if (out_local_address)
+-        *out_local_address = local_address;
+-    else
+-        g_free (local_address);
++        *out_local_address = g_steal_pointer (&local_address);
+     if (out_subnet)
+-        *out_subnet = subnet;
+-    else
+-        g_free (subnet);
+-
++        *out_subnet = g_steal_pointer (&subnet);
+     if (out_gateway_address)
+-        *out_gateway_address = gateway_address;
++        *out_gateway_address = g_steal_pointer (&gateway_address);
+     if (out_dns_primary_address)
+-        *out_dns_primary_address = dns_primary_address;
++        *out_dns_primary_address = g_steal_pointer (&dns_primary_address);
+     if (out_dns_secondary_address)
+-        *out_dns_secondary_address = dns_secondary_address;
++        *out_dns_secondary_address = g_steal_pointer (&dns_secondary_address);
+     return TRUE;
+ }
+ 
+-- 
+GitLab

_service:tar_scm:backport-modem-helpers-rework-the-CGEV-indication-parser.patch Added

@@ -0,0 +1,86 @@
+From 9bbc768666e8ca77abdda325c46588cbf8a4df64 Mon Sep 17 00:00:00 2001
+From: Aleksander Morgado <aleksandermj@chromium.org>
+Date: Wed, 24 Aug 2022 16:50:34 +0000
+Subject: PATCH modem-helpers: rework the +CGEV indication parser
+
+We setup all output variables with g_autofree and then use
+g_steal_pointer() to return the needed ones.
+---
+ src/mm-modem-helpers.c | 36 +++++++++++++++---------------------
+ 1 file changed, 15 insertions(+), 21 deletions(-)
+
+diff --git a/src/mm-modem-helpers.c b/src/mm-modem-helpers.c
+index 9c3b0ccdc..d71e1082c 100644
+--- a/src/mm-modem-helpers.c
++++ b/src/mm-modem-helpers.c
+@@ -3642,8 +3642,8 @@ mm_3gpp_parse_cgev_indication_pdp (const gchar  *str,
+     g_autoptr(GRegex)      r = NULL;
+     g_autoptr(GMatchInfo)  match_info = NULL;
+     GError                *inner_error = NULL;
+-    gchar                 *pdp_type = NULL;
+-    gchar                 *pdp_addr = NULL;
++    g_autofree gchar      *pdp_type = NULL;
++    g_autofree gchar      *pdp_addr = NULL;
+     guint                  cid = 0;
+ 
+     g_assert (type == MM_3GPP_CGEV_REJECT   ||
+@@ -3660,44 +3660,38 @@ mm_3gpp_parse_cgev_indication_pdp (const gchar  *str,
+ 
+     str = mm_strip_tag (str, "+CGEV:");
+     g_regex_match_full (r, str, strlen (str), 0, 0, &match_info, &inner_error);
+-    if (inner_error)
+-        goto out;
++    if (inner_error) {
++        g_propagate_error (error, inner_error);
++        return FALSE;
++    }
+ 
+     if (!g_match_info_matches (match_info)) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Couldn't match response");
+-        goto out;
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Couldn't match response");
++        return FALSE;
+     }
+ 
+     if (out_pdp_type && !(pdp_type = mm_get_string_unquoted_from_match_info (match_info, 1))) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing PDP type");
+-        goto out;
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing PDP type");
++        return FALSE;
+     }
+ 
+     if (out_pdp_addr && !(pdp_addr = mm_get_string_unquoted_from_match_info (match_info, 2))) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing PDP addr");
+-        goto out;
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing PDP addr");
++        return FALSE;
+     }
+ 
+     /* CID is optional */
+     if (out_cid &&
+         (g_match_info_get_match_count (match_info) >= 4) &&
+         !mm_get_uint_from_match_info (match_info, 3, &cid)) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing CID");
+-        goto out;
+-    }
+-
+-out:
+-    if (inner_error) {
+-        g_free (pdp_type);
+-        g_free (pdp_addr);
+-        g_propagate_error (error, inner_error);
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing CID");
+         return FALSE;
+     }
+ 
+     if (out_pdp_type)
+-        *out_pdp_type = pdp_type;
++        *out_pdp_type = g_steal_pointer (&pdp_type);
+     if (out_pdp_addr)
+-        *out_pdp_addr = pdp_addr;
++        *out_pdp_addr = g_steal_pointer (&pdp_addr);
+     if (out_cid)
+         *out_cid = cid;
+     return TRUE;
+-- 
+GitLab

_service:tar_scm:backport-modem-helpers-ublox-rework-UIPADDR-response-parser.patch Added

@@ -0,0 +1,115 @@
+From 8eeaba188272de496671763bc31f7fd592cb5ce0 Mon Sep 17 00:00:00 2001
+From: Aleksander Morgado <aleksandermj@chromium.org>
+Date: Wed, 24 Aug 2022 16:55:41 +0000
+Subject: PATCH modem-helpers-ublox: rework +UIPADDR response parser
+
+We setup all output variables with g_autofree and then use
+g_steal_pointer() to return the needed ones.
+---
+ plugins/ublox/mm-modem-helpers-ublox.c | 65 ++++++++++----------------
+ 1 file changed, 24 insertions(+), 41 deletions(-)
+
+diff --git a/plugins/ublox/mm-modem-helpers-ublox.c b/plugins/ublox/mm-modem-helpers-ublox.c
+index 5cc035422..84a7cf272 100644
+--- a/plugins/ublox/mm-modem-helpers-ublox.c
++++ b/plugins/ublox/mm-modem-helpers-ublox.c
+@@ -228,11 +228,11 @@ mm_ublox_parse_uipaddr_response (const gchar  *response,
+     g_autoptr(GMatchInfo)  match_info = NULL;
+     GError                *inner_error = NULL;
+     guint                  cid = 0;
+-    gchar                 *if_name = NULL;
+-    gchar                 *ipv4_address = NULL;
+-    gchar                 *ipv4_subnet = NULL;
+-    gchar                 *ipv6_global_address = NULL;
+-    gchar                 *ipv6_link_local_address = NULL;
++    g_autofree gchar      *if_name = NULL;
++    g_autofree gchar      *ipv4_address = NULL;
++    g_autofree gchar      *ipv4_subnet = NULL;
++    g_autofree gchar      *ipv6_global_address = NULL;
++    g_autofree gchar      *ipv6_link_local_address = NULL;
+ 
+     /* Response may be e.g.:
+      * +UIPADDR: 1,"ccinet0","5.168.120.13","255.255.255.0","",""
+@@ -245,61 +245,44 @@ mm_ublox_parse_uipaddr_response (const gchar  *response,
+     g_assert (r != NULL);
+ 
+     g_regex_match_full (r, response, strlen (response), 0, 0, &match_info, &inner_error);
+-    if (inner_error)
+-        goto out;
++    if (inner_error) {
++        g_propagate_error (error, inner_error);
++        return FALSE;
++    }
+ 
+     if (!g_match_info_matches (match_info)) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_INVALID_ARGS, "Couldn't match +UIPADDR response");
+-        goto out;
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_INVALID_ARGS, "Couldn't match +UIPADDR response");
++        return FALSE;
+     }
+ 
+     if (out_cid && !mm_get_uint_from_match_info (match_info, 1, &cid)) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing cid");
+-        goto out;
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing cid");
++        return FALSE;
+     }
+ 
+     if (out_if_name && !(if_name = mm_get_string_unquoted_from_match_info (match_info, 2))) {
+-        inner_error = g_error_new (MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing interface name");
+-        goto out;
++        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, "Error parsing interface name");
++        return FALSE;
+     }
+ 
+     /* Remaining strings are optional */
+-
+-    if (out_ipv4_address)
+-        ipv4_address = mm_get_string_unquoted_from_match_info (match_info, 3);
+-
+-    if (out_ipv4_subnet)
+-        ipv4_subnet = mm_get_string_unquoted_from_match_info (match_info, 4);
+-
+-    if (out_ipv6_global_address)
+-        ipv6_global_address = mm_get_string_unquoted_from_match_info (match_info, 5);
+-
+-    if (out_ipv6_link_local_address)
+-        ipv6_link_local_address = mm_get_string_unquoted_from_match_info (match_info, 6);
+-
+-out:
+-    if (inner_error) {
+-        g_free (if_name);
+-        g_free (ipv4_address);
+-        g_free (ipv4_subnet);
+-        g_free (ipv6_global_address);
+-        g_free (ipv6_link_local_address);
+-        g_propagate_error (error, inner_error);
+-        return FALSE;
+-    }
++    ipv4_address = mm_get_string_unquoted_from_match_info (match_info, 3);
++    ipv4_subnet = mm_get_string_unquoted_from_match_info (match_info, 4);
++    ipv6_global_address = mm_get_string_unquoted_from_match_info (match_info, 5);
++    ipv6_link_local_address = mm_get_string_unquoted_from_match_info (match_info, 6);
+ 
+     if (out_cid)
+         *out_cid = cid;
+     if (out_if_name)
+-        *out_if_name = if_name;
++        *out_if_name = g_steal_pointer (&if_name);
+     if (out_ipv4_address)
+-        *out_ipv4_address = ipv4_address;
++        *out_ipv4_address = g_steal_pointer (&ipv4_address);
+     if (out_ipv4_subnet)
+-        *out_ipv4_subnet = ipv4_subnet;
++        *out_ipv4_subnet = g_steal_pointer (&ipv4_subnet);
+     if (out_ipv6_global_address)
+-        *out_ipv6_global_address = ipv6_global_address;
++        *out_ipv6_global_address = g_steal_pointer (&ipv6_global_address);
+     if (out_ipv6_link_local_address)
+-        *out_ipv6_link_local_address = ipv6_link_local_address;
++        *out_ipv6_link_local_address = g_steal_pointer (&ipv6_link_local_address);
+     return TRUE;
+ }
+ 
+-- 
+GitLab

_service Changed

_service:extract_file:ModemManager-1.14.8.tar.xz Deleted

_service:extract_file:ModemManager.yaml Deleted

_service:tar_scm:ModemManager-1611798833.261aa49.tar -> _service:tar_scm:ModemManager-1.18.12.tar.xz Changed

_service:tar_scm:ModemManager.yaml Added