Bug 3615 – gkrellm-hddtemp-0.1.ebuild (new package)

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Imran Sher Rafique 2002-06-11 00:37:07 UTC]

A gkrellm plugin, using the hddtemp package (ebuild for that package supplied
seperately).

NOTE: The hddtemp binary needs to be installed 'setuid' for it to work for
non-root gkrellm instances. My solution to this involved the use of a
non-standard 'gkrellm' USE flag for the hddtemp package. If this solution is
rejected, then the hddtemp ebuild will have to install the main binary as
setuid, for it to work with this plugin.

[Comment 1 Imran Sher Rafique 2002-06-11 00:37:47 UTC]

Created attachment 1444 [details]
x11-misc/gkrellm-hddtemp/gkrellm-hddtemp-0.1.ebuild

ebuild

[Comment 2 Imran Sher Rafique 2002-06-11 00:38:21 UTC]

Created attachment 1445 [details]
x11-misc/gkrellm-hddtemp/ChangeLog

ChangeLog

[Comment 3 Imran Sher Rafique 2002-06-11 00:38:51 UTC]

Created attachment 1446 [details]
x11-misc/gkrellm-hddtemp/files/digest-gkrellm-hddtemp-0.1

md5 checksum

[Comment 4 George Shapovalov (RETIRED) 2002-06-22 00:25:50 UTC]

Hi Imran

Thanks for the submission!
Re: suid hddtemp: I would rather suggest modifying this ebuild to set suid bit
on hddtemp when this plugin is installed and remove on unmerge (see pkg_prerm
function). hddtemp is going to be installed first anyway - thus you are
guaranteed to get it correctly.
I am processing hddtemp now and I am going to delete the part that sets suid bit
from it. Could you please modify the plugin ebuild meanwhile?

Having an unnecessary suid bit would not be widely wellcomed for a security
reasons. Also, it would be nice if the plugin ebuild would output a message
about this suid bit being set upon completion (pkg_postinst function).

George

[Comment 5 Imran Sher Rafique 2002-06-22 01:14:26 UTC]

Created attachment 1682 [details]
amended ebuild

An amended ebuild, superseding the previously submitted ebuild. It seems that I
am not authorised to render previous attachments obsolete, so I will leave that
to the bug owner, George.

[Comment 6 Imran Sher Rafique 2002-06-22 01:15:38 UTC]

Made the requested changes. gkrellm-hddtemp ebuild now makes the hddtemp
'setuid' upon install of this plugin, and removes that bit upon un-install.

This will need to go hand-in-hand with removing the setuid section of the
hddtemp ebuild (as George stated).

I have already sent the amended ebuild as an attachment.

[Comment 7 George Shapovalov (RETIRED) 2002-06-25 16:11:47 UTC]

Hi Imran.

After the proper discussion the conclusion was reached that we should not set
any suids (in addition to standard well known apps) automatically in ebuilds. I
modified the ebuild to make an announcement that sysadmin will need to set
hddtemp suid in order to allow regular users run the plugin (and stripped actual
suid setting and resetting out).
I have committed the ebuild.

George