On 04/17/2013 08:08 PM, Patrick McHardy wrote:
Daniel Borkmann <dborkman@xxxxxxxxxx> schrieb:
On 04/17/2013 06:47 PM, Patrick McHardy wrote:
Signed-off-by: Patrick McHardy <kaber@xxxxxxxxx>
---
[...]
+Ring frames
+------------
+
+Each frames contain a frame header, consisting of a synchronization
word and some
+meta-data, and the message itself.
+
+Frame: [ header message ]
+
+The frame header is defined as follows:
+
+struct nl_mmap_hdr {
+ unsigned int nm_status;
+ unsigned int nm_len;
+ __u32 nm_group;
+ /* credentials */
+ __u32 nm_pid;
+ __u32 nm_uid;
+ __u32 nm_gid;
+};
Hm, just wondering, maybe it could be a safer path to make 'nm_status'
and 'nm_len' also of fixed width as the rest of nl_mmap_hdr, and as in
PF_PACKET's tpacket2_hdr and tpacket3_hdr ?
--
To unsubscribe from this list: send the line "unsubscribe
netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
The size of int is not going to change, but I agree that it is cleaner. I'll send a patch to change this once/if this is merged.
Sounds great, thanks !
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
