On Monday 2012-09-03 00:53, Eric Dumazet wrote:
>[PATCH] xt_LOG: take care of timewait sockets
>
>Sami Farin reported crashes in xt_LOG because it assumes skb->sk is a
>full blown socket.
>
>But with TCP early demux, we can have skb->sk pointing to a timewait
>socket.
>
>+static void dump_sk_uid_gid(struct sbuff *m, struct sock *sk)
>+{
>+ if (!sk || sk->sk_state == TCP_TIME_WAIT)
>+ return;
>+
>+ read_lock_bh(&sk->sk_callback_lock);
>+ if (sk->sk_socket && sk->sk_socket->file)
>+ sb_add(m, "UID=%u GID=%u ",
>+ sk->sk_socket->file->f_cred->fsuid,
>+ sk->sk_socket->file->f_cred->fsgid);
xt_owner.c is also using f_cred, so it might need the same,
does it not?
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
E1000-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel® Ethernet, visit
http://communities.intel.com/community/wired
