[lkml]   [2009]   [Jan]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH v3] tcp: splice as many packets as possible at once
    On Tue, Jan 27, 2009 at 10:35:11AM +0000, Jarek Poplawski wrote:
    > > > Yes, but ip_append_data() (and skb_append_datato_frags() for
    > > > NETIF_F_UFO only, so currently not a problem), uses this differently,
    > > > and these pages in sk->sk_sndmsg_page could leak or be used after
    > > > kfree. (I didn't track locking in these other places).
    > >
    > > It'll be freed when the socket is freed so that should be fine.
    > I don't think so: these places can overwrite sk->sk_sndmsg_page left
    > after tcp_sendmsg(), or skb_splice_bits() now, with NULL or a new
    > pointer without put_page() (they only reference copied chunks and
    > expect auto freeing). On the other hand, if tcp_sendmsg() reads after
    > them it could use a pointer after the page is freed, I guess.

    I wasn't referring to the first part of your sentence. That can't
    happen because they're only used for UDP sockets, this is a TCP

    Visit Openswan at
    Email: Herbert Xu ~{PmV>HI~} <>
    Home Page:
    PGP Key:

     \ /
      Last update: 2009-01-27 12:51    [W:0.027 / U:143.264 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site