Hello community, here is the log from the commit of package powertop for openSUSE:Factory checked in at 2015-02-12 10:19:40 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/powertop (Old) and /work/SRC/openSUSE:Factory/.powertop.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "powertop" Changes: -------- --- /work/SRC/openSUSE:Factory/powertop/powertop.changes 2014-12-19 09:36:57.000000000 +0100 +++ /work/SRC/openSUSE:Factory/.powertop.new/powertop.changes 2015-02-12 10:19:41.000000000 +0100 @@ -1,0 +2,7 @@ +Sun Feb 8 22:02:17 UTC 2015 - [email protected] + +- Add powertop-fix-crash-with-autotune.patch; Powertop crashes with + 'powertop --auto-tune' (fix bnc#915690) +- Split out translation files + +------------------------------------------------------------------- New: ---- powertop-fix-crash-with-autotune.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ powertop.spec ++++++ --- /var/tmp/diff_new_pack.kzwkma/_old 2015-02-12 10:19:42.000000000 +0100 +++ /var/tmp/diff_new_pack.kzwkma/_new 2015-02-12 10:19:42.000000000 +0100 @@ -1,7 +1,7 @@ # # spec file for package powertop # -# Copyright (c) 2014 SUSE LINUX Products GmbH, Nuernberg, Germany. +# Copyright (c) 2015 SUSE LINUX Products GmbH, Nuernberg, Germany. # # All modifications and additions to the file contributed by third parties # remain the property of their copyright owners, unless otherwise agreed @@ -26,6 +26,7 @@ Release: 0 Source: https://01.org/sites/default/files/downloads/%name/%name-%version.tar.gz Patch2: powertop-1.98-always-create-params.patch +Patch3: powertop-fix-crash-with-autotune.patch Patch5: powertop-no-date.patch BuildRequires: gcc-c++ BuildRequires: gettext @@ -36,6 +37,7 @@ BuildRequires: pkgconfig(libnl-genl-3.0) BuildRequires: pkgconfig(libpci) BuildRequires: pkgconfig(zlib) +Recommends: %{name}-lang BuildRoot: %{_tmppath}/%{name}-%{version}-build %description @@ -43,9 +45,12 @@ from your system and presents an overview of how well your laptop is doing in terms of power savings. +%lang_package + %prep %setup -q %patch2 -p1 +%patch3 -p1 %patch5 -p1 # Delete objects files left in tarball @@ -66,7 +71,7 @@ touch %{_localstatedir}/cache/powertop/saved_parameters.powertop touch %{_localstatedir}/cache/powertop/saved_results.powertop -%files -f %name.lang +%files %defattr(-,root,root) %doc COPYING README %dir %{_localstatedir}/cache/powertop @@ -75,4 +80,6 @@ %_sbindir/%name %{_mandir}/man8/powertop.8.gz +%files lang -f %{name}.lang + %changelog ++++++ powertop-fix-crash-with-autotune.patch ++++++ >From 6fd9ecebd3bc277bd91ce057f85f8e54b09bb0a9 Mon Sep 17 00:00:00 2001 From: Michael Mullin <[email protected]> Date: Thu, 22 Jan 2015 16:50:10 -0500 Subject: [PATCH] Powertop crashes with 'powertop --auto-tune' -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 Please see the following patch to address the reported crash - --- - From e994a414f619410877023fce35ade12d1cfce68d Mon Sep 17 00:00:00 2001 From: Michael Mullin <[email protected]> Date: Thu, 22 Jan 2015 16:33:31 -0500 Subject: [PATCH] If "dir" is closed early inside create_add_devfreq_devices() then there is a potential double free due to a second close during the cleanup phase in clear_all_devfreq() Signed-off-by: Michael Mullin <[email protected]> - --- src/devices/devfreq.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) --- src/devices/devfreq.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/devices/devfreq.cpp b/src/devices/devfreq.cpp index d2e56e3..f652a57 100644 --- a/src/devices/devfreq.cpp +++ b/src/devices/devfreq.cpp @@ -247,6 +247,7 @@ void create_all_devfreq_devices(void) fprintf(stderr, "Devfreq not enabled\n"); is_enabled = false; closedir(dir); + dir = NULL; return; } @@ -327,6 +328,8 @@ void clear_all_devfreq() } all_devfreq.clear(); /* close /sys/class/devfreq */ - if (dir != NULL) + if (dir != NULL) { closedir(dir); + dir = NULL; + } } -- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
