- Subject: Re: [PATCH 4/10] Fix leaking of kernel heap addresses in net/
- From: David Miller <davem@xxxxxxxxxxxxx>
- Date: Thu, 11 Nov 2010 18:49:02 -0800 (PST)
- Cc: socketcan@xxxxxxxxxxxx, kuznet@xxxxxxxxxxxxx, urs.thuermann@xxxxxxxxxxxxx, yoshfuji@xxxxxxxxxxxxxx, kaber@xxxxxxxxx, jmorris@xxxxxxxxx, remi.denis-courmont@xxxxxxxxx, pekkas@xxxxxxxxxx, sri@xxxxxxxxxx, vladislav.yasevich@xxxxxx, tj@xxxxxxxxxx, eric.dumazet@xxxxxxxxx, lizf@xxxxxxxxxxxxxx, joe@xxxxxxxxxxx, shemminger@xxxxxxxxxx, hadi@xxxxxxxxxxxx, ebiederm@xxxxxxxxxxxx, adobriyan@xxxxxxxxx, jpirko@xxxxxxxxxx, johannes.berg@xxxxxxxxx, daniel.lezcano@xxxxxxx, xemul@xxxxxxxxxx, socketcan-core@xxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, linux-sctp@xxxxxxxxxxxxxxx, torvalds@xxxxxxxxxxxxxxxxxxxx
- In-reply-to: <1289529269.3090.207.camel@Dan>
- References: <2129857903-1289528127-cardhu_decombobulator_blackberry.rim.net-1506931048-@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20101111.182939.258124014.davem@xxxxxxxxxxxxx> <1289529269.3090.207.camel@Dan>
From: Dan Rosenberg <drosenberg@xxxxxxxxxxxxx>
Date: Thu, 11 Nov 2010 21:34:29 -0500
>
>> I want whatever you replace it with to be equivalent for
>> object tracking purposes.
>
> In nearly all of the cases I fixed, the socket inode is already
> provided, which serves as a perfectly good unique identifier. Would you
> prefer I include that information twice?
The problem is that the socket inode is not available in a certain
subclass of cases, so the transformation is not equivalent.
Why not attack this at the heart of where your concern is, and hack
the %p format handling to do whatever it is you like instead of
patching code all over the tree?
--
To unsubscribe from this list: send the line "unsubscribe linux-sctp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Linux OMAP]
[Linux USB Devel]
[Video for Linux]
[Linux Audio Users]
[Photo]
[Yosemite News]
[Yosemite Photos]
[Free Online Dating]
[Linux Kernel]
[Linux SCSI]
[XFree86]