FS#33542 - [catalyst-utils] /etc/ati/authatieventsd.sh: Permission denied

Attached to Project: Community Packages
Opened by alexander (actics) - Thursday, 24 January 2013, 10:46 GMT
Last edited by Laurent Carlier (lordheavy) - Saturday, 20 April 2013, 18:40 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Laurent Carlier (lordheavy)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:

In my x86_64 arch, after installing catalyst-utils 13.1 i have a write to log (journalctl every second)

atieventsd[680]: sh: /etc/ati/authatieventsd.sh: Permission denied

after
# chmod +x /etc/ati/authatieventsd.sh
i have a

atieventsd[680]: /etc/ati/authatieventsd.sh: line 69: return: -1: invalid option

i rewrite line 69 to exit -1, and in journalctl is clear
This task depends upon

Closed by  Laurent Carlier (lordheavy)
Saturday, 20 April 2013, 18:40 GMT
Reason for closing:  Fixed
Comment by Vasil Yonkov (spirtbrat) - Monday, 18 February 2013, 22:17 GMT
Yes, /etc/ati/authatieventsd.sh should be installed with executable bit set, otherwise it does not execute :)

You get:
atieventsd[680]: /etc/ati/authatieventsd.sh: line 69: return: -1: invalid option
because the script somehow fails to determine any authorization key for X and hits line 69, which is:
return -1

Not finding an authorization key can happen if you start your X with something else than xdm, kdm, gdm or startx.
Or you haven't started X at all prior to executing the script.

Also, rewriting line 69 to exit -1 is not correct, as it exits the whole script, not only the function.
For bash, line 69 should be:
return -- -1

See this for more info:
http://gnu-bash.2382.n7.nabble.com/bug-return-doesn-t-accept-negative-numbers-tp2031p2032.html
Comment by Vasil Yonkov (spirtbrat) - Monday, 18 February 2013, 22:55 GMT
Since the other task was marked as duplicate - I will note here as well that authatieventsd.sh depends on

which
xorg-xauth
sysvinit-tools

packages in order to work and maybe they should be optional dependencies.
Comment by alexander (actics) - Tuesday, 19 February 2013, 10:00 GMT
I use a lxdm, therefore I must install a xdm, kdm, gdm or startx? Brr :)
Comment by Laurent Carlier (lordheavy) - Saturday, 02 March 2013, 10:05 GMT
@spirtbrat

which is part of base package
sysvinit-tools will be useless, i've replaced pidof with ps
xorg-xauth will be added as a dependency
Comment by Laurent Carlier (lordheavy) - Saturday, 02 March 2013, 10:57 GMT
Please test catalyst-utils-13.1-2
Comment by alexander (actics) - Sunday, 03 March 2013, 09:23 GMT
@lordheavy
Thanks! Test for bug on lxdm, slim and startx in my system is OK.
Comment by Vasil Yonkov (spirtbrat) - Sunday, 03 March 2013, 10:03 GMT
@lordheavy

In catalyst-utils-13.1-2 you directly include a modified version of
authatieventsd.sh instead of a patch for it. I agree this is more
convenient for the maintainer. This way, the fglrx-authfile-locations.patch
becomes unnecessary. You can include the changes in directly in the
redacted authatieventsd.sh.

Besides, since when the whole 'base' group is expected to be installed?
I thought this was the case only with 'base-devel'.
Comment by Vasil Yonkov (spirtbrat) - Tuesday, 05 March 2013, 07:42 GMT
As the package is installed now, the changes from 'fglrx-authfile-
locations.patch' do not take effect since it patches the original
'authatieventsd.sh' and after that the new, modified version of
'authatieventsd.sh' replaces it. 'fglrx-authfile-locations.patch' have
to be merged in the 'authatieventsd.sh' supplied with the package.
Comment by Laurent Carlier (lordheavy) - Saturday, 16 March 2013, 18:17 GMT
Please test catalyst-utils-13.1-4
Comment by Vasil Yonkov (spirtbrat) - Sunday, 17 March 2013, 09:47 GMT
Everything looks OK with catalyst-utils-13.1-5

Loading...