<div dir="ltr">Looks good to me. Do we need to port the same fix to libmbim?<div><br></div><div>Thanks,</div><div>Roshan Pius</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 9, 2014 at 11:11 AM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Roshan,<br>
<br>
Could you please review the following patches and tell me what you think?<br>
<br>
[PATCH 1/2] build: rename '--enable-qmi-proxy-username'...<br>
This one is just a rename of the configure option, so that it's more generic<br>
(i.e. not related just to the proxy).<br>
<br>
<br>
[PATCH 2/2] libqmi,utils: new internal __qmi_user_allowed() method<br>
This one consolidates a new helper method to check if the user is allowed or<br>
not. Please also note that your patch introduced a bug in qmi_proxy_open():<br>
when the user wasn't allowed you would return FALSE without the GError being<br>
set.<br>
<br>
--<br>
Aleksander<br>
<a href="https://aleksander.es" target="_blank">https://aleksander.es</a><br>
</blockquote></div><br></div>