From: Justin Pettit <jpettit@nicira.com>
Date: Thu, 7 Mar 2013 02:03:54 +0000 (-0800)
Subject: ofproto-dpif: Clarify that 'one_subfacet' may not always be valid.
X-Git-Tag: sliver-openvswitch-1.10.90-1~10^2~92
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=bcd516b795e1009daf18bb74e070ac510bc4c5d6;p=sliver-openvswitch.git

ofproto-dpif: Clarify that 'one_subfacet' may not always be valid.

Signed-off-by: Justin Pettit <jpettit@nicira.com>
---

diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index ff93dc3b4..228e28025 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -487,7 +487,9 @@ struct facet {
 
     /* Storage for a single subfacet, to reduce malloc() time and space
      * overhead.  (A facet always has at least one subfacet and in the common
-     * case has exactly one subfacet.) */
+     * case has exactly one subfacet.  However, 'one_subfacet' may not
+     * always be valid, since it could have been removed after newer
+     * subfacets were pushed onto the 'subfacets' list.) */
     struct subfacet one_subfacet;
 };