OSDN Git Service

[NET]: Make skb_seq_read unmap the last fragment
authorOlaf Kirch <olaf.kirch@oracle.com>
Sun, 24 Jun 2007 06:11:52 +0000 (23:11 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sun, 24 Jun 2007 06:11:52 +0000 (23:11 -0700)
commit5b5a60da281c767196427ce8144deae6ec46b389
tree02ac728c14eb8fa0bd49ac8ede6f15e760ddc3f3
parent515e06c4556bd8388db6b2bb2cd8859126932946
[NET]: Make skb_seq_read unmap the last fragment

Having walked through the entire skbuff, skb_seq_read would leave the
last fragment mapped.  As a consequence, the unwary caller would leak
kmaps, and proceed with preempt_count off by one. The only (kind of
non-intuitive) workaround is to use skb_seq_read_abort.

This patch makes sure skb_seq_read always unmaps frag_data after
having cycled through the skb's paged part.

Signed-off-by: Olaf Kirch <olaf.kirch@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/skbuff.c