git://git.onelab.eu / sliver-openvswitch.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: f0a6bdc) | patch
ofproto-dpif: Close race between processing packet_ins and checking seqno.
authorBen Pfaff <blp@nicira.com>
Wed, 30 Apr 2014 18:07:46 +0000 (11:07 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 30 Apr 2014 19:47:00 +0000 (12:47 -0700)
commitb53d5c33bc91bdd2e46c67aec3af04a4dc020e26
tree431874840f4c68c0b3c3b0e13db375a5e3b6b82atree | snapshot
parentf0a6bdc9f56a95157ca6ee3c1f763aed0e0dae08commit | diff
ofproto-dpif: Close race between processing packet_ins and checking seqno.

If a packet-in were to be queued, and the sequence number changed, after
grabbing the list of packet-ins, then the existing code could have gone to
sleep until something happened.  By grabbing the sequence number before
the list of packet-ins, we avoid this race.

Found by inspection.

Signed-off-by: Ben Pfaff <blp@nicira.com>
ofproto/ofproto-dpif.c diff | blob | history
sliver-openvswitch