<div dir="ltr">Hi Aleksander,<div><br></div><div>I'll port the same patch to libqmi & will send the patch in sometime. </div><div>How do we want to handle chown'ing the devices to the specific user? We'll need to edit the udev rules to assign the devices to the specified user. But, From what i can see the udev rules to detect the modems are currently owned by modemmanager & this particular username flag is residing in libmbim/libqmi.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 7, 2014 at 10:56 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"><span class="">On Fri, Nov 7, 2014 at 7:39 PM, Dan Williams <<a href="mailto:dcbw@redhat.com">dcbw@redhat.com</a>> wrote:<br>
> On Fri, 2014-11-07 at 09:20 -0800, Roshan Pius wrote:<br>
>> Currently, the MBIM proxy process assumes that it is run as root user and<br>
>> that all incoming client connection users are also root.<br>
>> However, it's not always preferable to run the MBIM proxy as root for<br>
>> security reasons. On some platforms, the MBIM proxy could be constrained<br>
>> to run as a less-privileged user and specially granted the permission to<br>
>> access the MBIM device. So, adding a compile time flag in libmbim to check<br>
>> for the specified user, rather than assume it to be the root user. If the flag is<br>
>> not sent, it'll revert to the existing behaviour of checking for<br>
>> user=root(i.e UID=0)<br>
><br>
> Looks OK to me.<br>
<br>
<br>
</span>Great!<br>
<br>
Roshan, will you now write the patch to chown the devices to the<br>
specific user given in that configure flag? That would be superb.<br>
<br>
I'll actually test the patch in the following days and merge it afterwards.<br>
<br>
Plus, anyone willing to do the same for libqmi? :)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Aleksander<br>
<a href="https://aleksander.es" target="_blank">https://aleksander.es</a><br>
</font></span></blockquote></div><br></div>