linux-kernel - Re: [PATCH] taskstats: Use better ifdef for alignment

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 8 Jan 2011 23:42:00 +0100
From:	Florian Mickler <florian@...kler.org>
To:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Cc:	Dan Carpenter <error27@...il.com>, Jeff Mahoney <jeffm@...e.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	"David S. Miller" <davem@...emloft.net>, balbir@...ux.vnet.ibm.com,
	guichaz@...il.com
Subject: Re: [PATCH] taskstats: Use better ifdef for alignment

On Sun, 2 Jan 2011 13:17:47 +0100
Florian Mickler <florian@...kler.org> wrote:

> On Sat, 1 Jan 2011 19:51:35 +0300
> Dan Carpenter <error27@...il.com> wrote:
> 
> > On Sat, Jan 01, 2011 at 05:19:46PM +0100, Florian Mickler wrote:
> > > Is there already a patch available or integrated into iotop which
> > > fixes this? I'd think that if the kernel could wait on fixed iotop's to
> > > be distributed it would be easier to sell the breakage on bugzilla &
> > > co...

Ok, I sent a fix for the iotop parsing to Guillaume, but didn't receive
any reaction... maybe someone here find's it useful... 

commit 392f7e7455c94cdc64aeb0353a79d22b4780687c
Author: Florian Mickler <florian@...kler.org>
Date:   Tue Jan 4 12:03:25 2011 +0100

    fix parsing of netlink messages
    
    The linux kernel may enhance the message with fields to pad the taskstats struct
    (commit 4be2c95d1f ["taskstats: pad taskstats netlink
    response for aligment issues on ia64"] in Linus tree).
    So now we honour the flexible format and extract the field we are
    interested in.

diff --git a/iotop/data.py b/iotop/data.py
index 1164344..c75c190 100644
--- a/iotop/data.py
+++ b/iotop/data.py
@@ -136,6 +136,7 @@ TASKSTATS_CMD_GET = 1
 TASKSTATS_CMD_ATTR_PID = 1
 TASKSTATS_TYPE_AGGR_PID = 4
 TASKSTATS_TYPE_PID = 1
+TASKSTATS_TYPE_STATS = 3
 
 class TaskStatsNetlink(object):
     # Keep in sync with format_stats() and pinfo.did_some_io()
@@ -163,19 +164,23 @@ class TaskStatsNetlink(object):
         if len(reply.payload) < 292:
             # Short reply
             return
-        reply_length, reply_type = struct.unpack('HH', reply.payload[4:8])
-        assert reply_length >= 288
-        assert reply_type == TASKSTATS_TYPE_AGGR_PID
-
-        pid_length, pid_type = struct.unpack('HH', reply.payload[8:12])
-        assert pid_type == TASKSTATS_TYPE_PID
-
-        taskstats_start = 4 + 4 + pid_length + 4
-        taskstats_data = reply.payload[taskstats_start:]
+	reply.payload = reply.payload[4:]
+	data = get_data(TASKSTATS_TYPE_AGGR_PID, reply.payload)
+	assert( len(data) > 0)
+	taskstats_data = get_data(TASKSTATS_TYPE_STATS, data)
         taskstats_version = struct.unpack('H', taskstats_data[:2])[0]
         assert taskstats_version >= 4
         return Stats(taskstats_data)
 
+def get_data(field_type, payload):
+	while len(payload) > 3:
+		reply_length, reply_type = struct.unpack('HH', payload[:4])
+		if (reply_type == field_type):
+			break
+		payload = payload[reply_length:]
+
+	return payload[4:reply_length]
+
 #
 # PIDs manipulations
 #


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/