Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 9 Feb 2015 00:04:13 +0000
From:      "davide (Davide Italiano)" <phabric-noreply@FreeBSD.org>
To:        freebsd-net@freebsd.org
Subject:   [Differential] [Updated, 12 lines] D1805: [sockbuf] Don't access fields directly, use accessor functions
Message-ID:  <392d01c0d7111c7fadaacd9d2e1db1cd@localhost.localdomain>
In-Reply-To: <differential-rev-PHID-DREV-go3bp25pr252wjmukaeb-req@FreeBSD.org>
References:  <differential-rev-PHID-DREV-go3bp25pr252wjmukaeb-req@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
davide updated this revision to Diff 3698.
davide added a comment.

Updated removing the last place where this was used. I was not certain about it before so I previously left it as-is, but it looks like in this case order doesn't matter considering we're holding the socket buffer lock across the two calls.

CHANGES SINCE LAST UPDATE
  https://reviews.freebsd.org/D1805?vs=3695&id=3698

REVISION DETAIL
  https://reviews.freebsd.org/D1805

AFFECTED FILES
  sys/kern/uipc_socket.c

To: davide, kmacy, np, lstewart, rrs, rwatson
Cc: freebsd-net



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?392d01c0d7111c7fadaacd9d2e1db1cd>