<div dir="ltr"><div>&gt; Thanks, I applied this with number of changes.<br>&gt; so I replaced the addition here by<br>
tracking whether the vif was added (that &quot;create&quot; parameter on the add<br>
command).<br></div>This looks better. Thanks Jouni!<br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><br></div><div><b style="color:rgb(102,102,102)"><font face="&#39;courier new&#39;, monospace"><br></font></b></div><div><font face="comic sans ms, sans-serif" color="#000000"><span style="background-color:rgb(243,243,243)"><b>- JJ</b></span></font></div><div><br></div><div><b><font face="&#39;courier new&#39;, monospace"><br></font></b></div></div></div></div>
<br><div class="gmail_quote">On Fri, Mar 27, 2015 at 2:51 AM, Jouni Malinen <span dir="ltr">&lt;<a href="mailto:j@w1.fi" target="_blank">j@w1.fi</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Mar 20, 2015 at 04:09:50PM +0530, Jithu Jance wrote:<br>
&gt; Please find the modified patch below. Kindly see whether this is fine.<br>
&gt;<br>
&gt; Adding support for virtual interface creation and deletion<br>
&gt; for interface_add and interface_remove commands respectively<br>
&gt; (via optional argument)<br>
<br>
</span>Thanks, I applied this with number of changes.<br>
<span class=""><br>
&gt; -      * &lt;ifname&gt;TAB&lt;confname&gt;TAB&lt;driver&gt;TAB&lt;ctrl_interface&gt;TAB&lt;driver_param&gt;<br>
&gt; +      * &lt;ifname&gt;TAB[&lt;create&gt;TAB]&lt;confname&gt;TAB&lt;driver&gt;TAB&lt;ctrl_interface&gt;TAB&lt;driver_param&gt;<br>
&gt;        * TAB&lt;bridge_ifname&gt;<br>
<br>
</span>This does not work very well since &quot;create&quot; is a valid prefix for the<br>
existing confname parameter. I moved &quot;TAB&lt;create&gt;&quot; to the end of the<br>
command.<br>
<span class=""><br>
&gt; @@ -8564,13 +8588,35 @@ static int wpa_supplicant_global_iface_remove(struct wpa_global *global,<br>
</span><span class="">&gt; +      * &lt;ifname&gt;TAB[&lt;delete&gt;]<br>
<br>
</span>And this does not look desired, so I replaced the addition here by<br>
tracking whether the vif was added (that &quot;create&quot; parameter on the add<br>
command).<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Jouni Malinen                                            PGP id EFC895FA<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
HostAP mailing list<br>
<a href="mailto:HostAP@lists.shmoo.com">HostAP@lists.shmoo.com</a><br>
<a href="http://lists.shmoo.com/mailman/listinfo/hostap" target="_blank">http://lists.shmoo.com/mailman/listinfo/hostap</a><br>
</div></div></blockquote></div><br></div>