From camerontnorman at gmail.com Fri Feb 13 20:39:15 2015 Content-Type: multipart/mixed; boundary="===============7701123530086677994==" MIME-Version: 1.0 From: Cameron Norman To: devel at ovirt.org Subject: [ovirt-devel] ovirt-guest-agent: Hardcoded uid/gid: okay to remove? Date: Fri, 13 Feb 2015 17:39:12 -0800 Message-ID: <1423877952.28334.0@smtp.gmail.com> --===============7701123530086677994== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Hello, In the RPM and debian packaging for ovirt-guest-agent, the UID/GID = numbers are hardcoded. I need to remove this to fix a policy violation = in Debian. What are the implications of making the uid/gid's = dynamically allocated, i.e. why were they hardcoded in the first place? Thank you all, -- Cameron Norman --===============7701123530086677994== Content-Type: text/html MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.html" SGVsbG8sPGRpdj48YnI+PC9kaXY+PGRpdj5JbiB0aGUgUlBNIGFuZCBkZWJpYW4gcGFja2FnaW5n IGZvciBvdmlydC1ndWVzdC1hZ2VudCwgdGhlIFVJRC9HSUQgbnVtYmVycyBhcmUgaGFyZGNvZGVk LiBJIG5lZWQgdG8gcmVtb3ZlIHRoaXMgdG8gZml4IGEgcG9saWN5IHZpb2xhdGlvbiBpbiBEZWJp YW4uIFdoYXQgYXJlIHRoZSBpbXBsaWNhdGlvbnMgb2YgbWFraW5nIHRoZSB1aWQvZ2lkJ3MgZHlu YW1pY2FsbHkgYWxsb2NhdGVkLCBpLmUuIHdoeSB3ZXJlIHRoZXkgaGFyZGNvZGVkIGluIHRoZSBm aXJzdCBwbGFjZT88L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2PlRoYW5rIHlvdSBhbGwsPC9kaXY+ PGRpdj4tLTwvZGl2PjxkaXY+Q2FtZXJvbiBOb3JtYW48L2Rpdj4= --===============7701123530086677994==-- From vfeenstr at redhat.com Wed Feb 18 05:58:40 2015 Content-Type: multipart/mixed; boundary="===============6603744889512525055==" MIME-Version: 1.0 From: Vinzenz Feenstra To: devel at ovirt.org Subject: Re: [ovirt-devel] ovirt-guest-agent: Hardcoded uid/gid: okay to remove? Date: Wed, 18 Feb 2015 11:58:36 +0100 Message-ID: <54E4705C.9070606@redhat.com> In-Reply-To: 1423877952.28334.0@smtp.gmail.com --===============6603744889512525055== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable This is a multi-part message in MIME format. --------------040902040800050005030100 Content-Type: text/plain; charset=3Dwindows-1252; format=3Dflowed Content-Transfer-Encoding: 7bit On 02/14/2015 02:39 AM, Cameron Norman wrote: > Hello, Hi, Well I think there shouldn't be anything broken if you would remove it, = as we do reference everything by name everywhere. The reason for hard-coding it is that we have it on fedora and RHEL in = the setup package See: = https://git.fedorahosted.org/cgit/setup.git/tree/uidgid It's something like a well-known UID/GID when specified there. Regards, > > In the RPM and debian packaging for ovirt-guest-agent, the UID/GID = > numbers are hardcoded. I need to remove this to fix a policy violation = > in Debian. What are the implications of making the uid/gid's = > dynamically allocated, i.e. why were they hardcoded in the first place? > > Thank you all, > -- > Cameron Norman > > > _______________________________________________ > Devel mailing list > Devel(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/devel -- = Regards, Vinzenz Feenstra | Senior Software Engineer RedHat Engineering Virtualization R & D Phone: +420 532 294 625 IRC: vfeenstr or evilissimo Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com --------------040902040800050005030100 Content-Type: text/html; charset=3Dwindows-1252 Content-Transfer-Encoding: 7bit
On 02/14/2015 02:39 AM, Cameron Norman wrote:
Hello,
Hi,

Well I think there shouldn't be anything broken if you would remove it, as we do reference everything by name everywhere.
The reason for hard-coding it is that we have it on fedora and RHEL in the setup package See: https://git.fedorahosted.org/cgit/setup.git/tr= ee/uidgid
It's something like a well-known UID/GID when specified there.

Regards,

In the RPM and debian packaging for ovirt-guest-agent, the UID/GID numbers are hardcoded. I need to remove this to fix a policy violation in Debian. What are the implications of making the uid/gid's dynamically allocated, i.e. why were they hardcoded in the first place?

Thank you all,
--
Cameron Norman


_______________________________________________
Devel mailing list
Dev=
el(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel


-- =

Regards,

Vinzenz Feenstra | Senior Software Engineer
RedHat Engineering Virtualization R & D
Phone: +420 532 294 625
IRC: vfeenstr or evilissimo

Better technology. Faster innovation. Powered by community collaboration.
See how it works at redhat.com
--------------040902040800050005030100-- --===============6603744889512525055== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" VGhpcyBpcyBhIG11bHRpLXBhcnQgbWVzc2FnZSBpbiBNSU1FIGZvcm1hdC4KLS0tLS0tLS0tLS0t LS0wNDA5MDIwNDA4MDAwNTAwMDUwMzAxMDAKQ29udGVudC1UeXBlOiB0ZXh0L3BsYWluOyBjaGFy c2V0PXdpbmRvd3MtMTI1MjsgZm9ybWF0PWZsb3dlZApDb250ZW50LVRyYW5zZmVyLUVuY29kaW5n OiA3Yml0CgpPbiAwMi8xNC8yMDE1IDAyOjM5IEFNLCBDYW1lcm9uIE5vcm1hbiB3cm90ZToKPiBI ZWxsbywKSGksCgpXZWxsIEkgdGhpbmsgdGhlcmUgc2hvdWxkbid0IGJlIGFueXRoaW5nIGJyb2tl biBpZiB5b3Ugd291bGQgcmVtb3ZlIGl0LCAKYXMgd2UgZG8gcmVmZXJlbmNlIGV2ZXJ5dGhpbmcg YnkgbmFtZSBldmVyeXdoZXJlLgpUaGUgcmVhc29uIGZvciBoYXJkLWNvZGluZyBpdCBpcyB0aGF0 IHdlIGhhdmUgaXQgb24gZmVkb3JhIGFuZCBSSEVMIGluIAp0aGUgc2V0dXAgcGFja2FnZSBTZWU6 IApodHRwczovL2dpdC5mZWRvcmFob3N0ZWQub3JnL2NnaXQvc2V0dXAuZ2l0L3RyZWUvdWlkZ2lk Ckl0J3Mgc29tZXRoaW5nIGxpa2UgYSB3ZWxsLWtub3duIFVJRC9HSUQgd2hlbiBzcGVjaWZpZWQg dGhlcmUuCgpSZWdhcmRzLAo+Cj4gSW4gdGhlIFJQTSBhbmQgZGViaWFuIHBhY2thZ2luZyBmb3Ig b3ZpcnQtZ3Vlc3QtYWdlbnQsIHRoZSBVSUQvR0lEIAo+IG51bWJlcnMgYXJlIGhhcmRjb2RlZC4g SSBuZWVkIHRvIHJlbW92ZSB0aGlzIHRvIGZpeCBhIHBvbGljeSB2aW9sYXRpb24gCj4gaW4gRGVi aWFuLiBXaGF0IGFyZSB0aGUgaW1wbGljYXRpb25zIG9mIG1ha2luZyB0aGUgdWlkL2dpZCdzIAo+ IGR5bmFtaWNhbGx5IGFsbG9jYXRlZCwgaS5lLiB3aHkgd2VyZSB0aGV5IGhhcmRjb2RlZCBpbiB0 aGUgZmlyc3QgcGxhY2U/Cj4KPiBUaGFuayB5b3UgYWxsLAo+IC0tCj4gQ2FtZXJvbiBOb3JtYW4K Pgo+Cj4gX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KPiBE ZXZlbCBtYWlsaW5nIGxpc3QKPiBEZXZlbEBvdmlydC5vcmcKPiBodHRwOi8vbGlzdHMub3ZpcnQu b3JnL21haWxtYW4vbGlzdGluZm8vZGV2ZWwKCgotLSAKUmVnYXJkcywKClZpbnplbnogRmVlbnN0 cmEgfCBTZW5pb3IgU29mdHdhcmUgRW5naW5lZXIKUmVkSGF0IEVuZ2luZWVyaW5nIFZpcnR1YWxp emF0aW9uIFIgJiBEClBob25lOiArNDIwIDUzMiAyOTQgNjI1CklSQzogdmZlZW5zdHIgb3IgZXZp bGlzc2ltbwoKQmV0dGVyIHRlY2hub2xvZ3kuIEZhc3RlciBpbm5vdmF0aW9uLiBQb3dlcmVkIGJ5 IGNvbW11bml0eSBjb2xsYWJvcmF0aW9uLgpTZWUgaG93IGl0IHdvcmtzIGF0IHJlZGhhdC5jb20K CgotLS0tLS0tLS0tLS0tLTA0MDkwMjA0MDgwMDA1MDAwNTAzMDEwMApDb250ZW50LVR5cGU6IHRl eHQvaHRtbDsgY2hhcnNldD13aW5kb3dzLTEyNTIKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzog N2JpdAoKPGh0bWw+CiAgPGhlYWQ+CiAgICA8bWV0YSBjb250ZW50PSJ0ZXh0L2h0bWw7IGNoYXJz ZXQ9d2luZG93cy0xMjUyIgogICAgICBodHRwLWVxdWl2PSJDb250ZW50LVR5cGUiPgogIDwvaGVh ZD4KICA8Ym9keSB0ZXh0PSIjMDAwMDAwIiBiZ2NvbG9yPSIjRkZGRkZGIj4KICAgIDxkaXYgY2xh c3M9Im1vei1jaXRlLXByZWZpeCI+T24gMDIvMTQvMjAxNSAwMjozOSBBTSwgQ2FtZXJvbiBOb3Jt YW4KICAgICAgd3JvdGU6PGJyPgogICAgPC9kaXY+CiAgICA8YmxvY2txdW90ZSBjaXRlPSJtaWQ6 MTQyMzg3Nzk1Mi4yODMzNC4wQHNtdHAuZ21haWwuY29tIiB0eXBlPSJjaXRlIj5IZWxsbyw8L2Js b2NrcXVvdGU+CiAgICBIaSw8YnI+CiAgICA8YnI+CiAgICBXZWxsIEkgdGhpbmsgdGhlcmUgc2hv dWxkbid0IGJlIGFueXRoaW5nIGJyb2tlbiBpZiB5b3Ugd291bGQgcmVtb3ZlCiAgICBpdCwgYXMg d2UgZG8gcmVmZXJlbmNlIGV2ZXJ5dGhpbmcgYnkgbmFtZSBldmVyeXdoZXJlLjxicj4KICAgIFRo ZSByZWFzb24gZm9yIGhhcmQtY29kaW5nIGl0IGlzIHRoYXQgd2UgaGF2ZSBpdCBvbiBmZWRvcmEg YW5kIFJIRUwKICAgIGluIHRoZSBzZXR1cCBwYWNrYWdlIFNlZToKICAgIDxhIGNsYXNzPSJtb3ot dHh0LWxpbmstZnJlZXRleHQiIGhyZWY9Imh0dHBzOi8vZ2l0LmZlZG9yYWhvc3RlZC5vcmcvY2dp dC9zZXR1cC5naXQvdHJlZS91aWRnaWQiPmh0dHBzOi8vZ2l0LmZlZG9yYWhvc3RlZC5vcmcvY2dp dC9zZXR1cC5naXQvdHJlZS91aWRnaWQ8L2E+PGJyPgogICAgSXQncyBzb21ldGhpbmcgbGlrZSBh IHdlbGwta25vd24gVUlEL0dJRCB3aGVuIHNwZWNpZmllZCB0aGVyZS48YnI+CiAgICA8YnI+CiAg ICBSZWdhcmRzLDxicj4KICAgIDxibG9ja3F1b3RlIGNpdGU9Im1pZDoxNDIzODc3OTUyLjI4MzM0 LjBAc210cC5nbWFpbC5jb20iIHR5cGU9ImNpdGUiPgogICAgICA8ZGl2Pjxicj4KICAgICAgPC9k aXY+CiAgICAgIDxkaXY+SW4gdGhlIFJQTSBhbmQgZGViaWFuIHBhY2thZ2luZyBmb3Igb3ZpcnQt Z3Vlc3QtYWdlbnQsIHRoZQogICAgICAgIFVJRC9HSUQgbnVtYmVycyBhcmUgaGFyZGNvZGVkLiBJ IG5lZWQgdG8gcmVtb3ZlIHRoaXMgdG8gZml4IGEKICAgICAgICBwb2xpY3kgdmlvbGF0aW9uIGlu IERlYmlhbi4gV2hhdCBhcmUgdGhlIGltcGxpY2F0aW9ucyBvZiBtYWtpbmcKICAgICAgICB0aGUg dWlkL2dpZCdzIGR5bmFtaWNhbGx5IGFsbG9jYXRlZCwgaS5lLiB3aHkgd2VyZSB0aGV5CiAgICAg ICAgaGFyZGNvZGVkIGluIHRoZSBmaXJzdCBwbGFjZT88L2Rpdj4KICAgICAgPGRpdj48YnI+CiAg ICAgIDwvZGl2PgogICAgICA8ZGl2PlRoYW5rIHlvdSBhbGwsPC9kaXY+CiAgICAgIDxkaXY+LS08 L2Rpdj4KICAgICAgPGRpdj5DYW1lcm9uIE5vcm1hbjwvZGl2PgogICAgICA8YnI+CiAgICAgIDxm aWVsZHNldCBjbGFzcz0ibWltZUF0dGFjaG1lbnRIZWFkZXIiPjwvZmllbGRzZXQ+CiAgICAgIDxi cj4KICAgICAgPHByZSB3cmFwPSIiPl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fCkRldmVsIG1haWxpbmcgbGlzdAo8YSBjbGFzcz0ibW96LXR4dC1saW5rLWFi YnJldmlhdGVkIiBocmVmPSJtYWlsdG86RGV2ZWxAb3ZpcnQub3JnIj5EZXZlbEBvdmlydC5vcmc8 L2E+CjxhIGNsYXNzPSJtb3otdHh0LWxpbmstZnJlZXRleHQiIGhyZWY9Imh0dHA6Ly9saXN0cy5v dmlydC5vcmcvbWFpbG1hbi9saXN0aW5mby9kZXZlbCI+aHR0cDovL2xpc3RzLm92aXJ0Lm9yZy9t YWlsbWFuL2xpc3RpbmZvL2RldmVsPC9hPjwvcHJlPgogICAgPC9ibG9ja3F1b3RlPgogICAgPGJy PgogICAgPGJyPgogICAgPHByZSBjbGFzcz0ibW96LXNpZ25hdHVyZSIgY29scz0iNzIiPi0tIApS ZWdhcmRzLAoKVmluemVueiBGZWVuc3RyYSB8IFNlbmlvciBTb2Z0d2FyZSBFbmdpbmVlcgpSZWRI YXQgRW5naW5lZXJpbmcgVmlydHVhbGl6YXRpb24gUiAmYW1wOyBEClBob25lOiArNDIwIDUzMiAy OTQgNjI1CklSQzogdmZlZW5zdHIgb3IgZXZpbGlzc2ltbwoKQmV0dGVyIHRlY2hub2xvZ3kuIEZh c3RlciBpbm5vdmF0aW9uLiBQb3dlcmVkIGJ5IGNvbW11bml0eSBjb2xsYWJvcmF0aW9uLgpTZWUg aG93IGl0IHdvcmtzIGF0IHJlZGhhdC5jb208L3ByZT4KICA8L2JvZHk+CjwvaHRtbD4KCi0tLS0t LS0tLS0tLS0tMDQwOTAyMDQwODAwMDUwMDA1MDMwMTAwLS0K --===============6603744889512525055==--