tags 682388 + pending
thanks

Dear maintainer,

I've prepared an NMU for powertop (versioned as 2.0-0.3) and
uploaded it to DELAYED/2. Please feel free to tell me if I
should delay it longer.

Regards.

-- 
Jonathan Wiltshire                                      j...@debian.org
Debian Developer                         http://people.debian.org/~jmw

4096R: 0xD3524C51 / 0A55 B7C5 1223 3942 86EC  74C3 5394 479D D352 4C51

<directhex> i have six years of solaris sysadmin experience, from
            8->10. i am well qualified to say it is made from bonghits
                        layered on top of bonghits
diff -Nru powertop-2.0/debian/changelog powertop-2.0/debian/changelog
--- powertop-2.0/debian/changelog	2012-06-30 15:13:48.000000000 +0100
+++ powertop-2.0/debian/changelog	2013-04-21 00:38:17.000000000 +0100
@@ -1,3 +1,11 @@
+powertop (2.0-0.3) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Add patch long-process-names-nullterm.patch to prevent a crash
+    on paticularly long process names (Closes: #682388)
+
+ -- Jonathan Wiltshire <j...@debian.org>  Sun, 21 Apr 2013 00:38:10 +0100
+
 powertop (2.0-0.2) unstable; urgency=low
 
   * Non-maintainer upload.
diff -Nru powertop-2.0/debian/patches/long-process-names-nullterm.patch powertop-2.0/debian/patches/long-process-names-nullterm.patch
--- powertop-2.0/debian/patches/long-process-names-nullterm.patch	1970-01-01 01:00:00.000000000 +0100
+++ powertop-2.0/debian/patches/long-process-names-nullterm.patch	2013-04-21 00:37:22.000000000 +0100
@@ -0,0 +1,26 @@
+Description: null-terminate long names of processes, otherwise a crash
+ can occur
+Author: Mikko Rapeli <mikko.rap...@iki.fi>
+Bug-Debian: http://bugs.debian.org/682388
+Forwarded: not-needed
+Last-Update: 20130421
+
+--- powertop-2.0.orig/src/process/process.cpp
++++ powertop-2.0/src/process/process.cpp
+@@ -86,7 +86,7 @@
+ 
+ process::process(const char *_comm, int _pid, int _tid) : power_consumer()
+ {
+-	char line[4096];
++	char line[4097];
+ 	ifstream file;
+ 
+ 	strcpy(comm, _comm);
+@@ -103,6 +103,7 @@
+ 		file.open(line);
+ 		while (file) {
+ 			file.getline(line, 4096);
++			line[4096] = '\0';
+ 			if (strstr(line, "Tgid")) {
+ 				char *c;
+ 				c = strchr(line, ':');
diff -Nru powertop-2.0/debian/patches/series powertop-2.0/debian/patches/series
--- powertop-2.0/debian/patches/series	2012-06-30 14:58:08.000000000 +0100
+++ powertop-2.0/debian/patches/series	2013-04-21 00:34:30.000000000 +0100
@@ -0,0 +1 @@
+long-process-names-nullterm.patch

Attachment: signature.asc
Description: Digital signature

Reply via email to