Thanks for the comments, I&#39;ve folded them in.<br><br><div class="gmail_quote">On Thu, Dec 27, 2012 at 11:34 AM, Ben Pfaff <span dir="ltr">&lt;<a href="mailto:blp@nicira.com" target="_blank">blp@nicira.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Wed, Dec 26, 2012 at 05:16:42PM -0800, Ethan Jackson wrote:<br>
&gt; Theoretically, it&#39;s possible for netdev_get_status() to be called<br>
&gt; on a netdev-vport which hasn&#39;t had it&#39;s configuration set yet.  In<br>
<br>
</div>s/it&#39;s/its/<br>
<div class="im"><br>
&gt; this case, netdev-vport would dereference a null pointer.  This<br>
&gt; problem was found by Jesse Gross &lt;<a href="mailto:jesse@nicira.com">jesse@nicira.com</a>&gt; in review.<br>
<br>
</div>I usually would write this as:<br>
Reported-by: Jesse Gross &lt;<a href="mailto:jesse@nicira.com">jesse@nicira.com</a>&gt;<br>
<div class="im"><br>
&gt; Signed-off-by: Ethan Jackson &lt;<a href="mailto:ethan@nicira.com">ethan@nicira.com</a>&gt;<br>
&gt; ---<br>
&gt;  lib/netdev-vport.c |    6 ++++++<br>
&gt;  1 file changed, 6 insertions(+)<br>
&gt;<br>
&gt; diff --git a/lib/netdev-vport.c b/lib/netdev-vport.c<br>
&gt; index e50103b..9bb5d36 100644<br>
&gt; --- a/lib/netdev-vport.c<br>
&gt; +++ b/lib/netdev-vport.c<br>
&gt; @@ -460,6 +460,12 @@ netdev_vport_get_tnl_iface(const struct netdev *netdev)<br>
&gt;      static char name[IFNAMSIZ];<br>
&gt;<br>
&gt;      ndv = netdev_dev_vport_cast(netdev_get_dev(netdev));<br>
&gt; +    if (!ndv-&gt;options) {<br>
&gt; +        /* Race condition when &#39;ndv&#39; was created, but did not have it&#39;s<br>
<br>
</div>s/it&#39;s/its/<br>
<div class="HOEnZb"><div class="h5"><br>
&gt; +         * configuration set yet. */<br>
&gt; +        return NULL;<br>
&gt; +    }<br>
&gt; +<br>
</div></div></blockquote></div><br>