Bug 1311547

Summary:	Wrong perms on systemd service file
Product:	Red Hat Enterprise Linux 7	Reporter:	Mario Trangoni <mjtrangoni>
Component:	wpa_supplicant	Assignee:	Lubomir Rintel <lrintel>
Status:	CLOSED DUPLICATE	QA Contact:	Ken Benoit <kbenoit>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	7.2	CC:	aloughla, dcbw, extras-qa, fedora, lrintel, mathieu-acct, mjtrangoni, orion, pomidorabelisima, rodd, rvcsaba, sknauss, somlo, terje.rosten, zbyszek
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	803980	Environment:
Last Closed:	2016-03-17 09:09:57 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Mario Trangoni 2016-02-24 12:50:16 UTC

+++ This bug was initially created as a clone of Bug #803980 +++

Description of problem:

 /lib/systemd/system/wpa_supplicant.service

should have not have execute bits set, it's a config file.

change 0755 in
  install -D -m 0755 %{SOURCE3} %{buildroot}/%{_unitdir}/%{name}.service
to 0644.

--- Additional comment from Fedora End Of Life on 2013-04-03 10:23:46 EDT ---

This bug appears to have been reported against 'rawhide' during the Fedora 19 development cycle.
Changing version to '19'.

(As we did not run this process for some time, it could affect also pre-Fedora 19 development
cycle bugs. We are very sorry. It will help us with cleanup during Fedora 19 End Of Life. Thank you.)

More information and reason for this action is here:
https://fedoraproject.org/wiki/BugZappers/HouseKeeping/Fedora19

--- Additional comment from Terje Røsten on 2013-11-24 04:02:53 EST ---

Any progress here, can't be that hard to fix this?

There are several other easy to fix bugs against this components, do you
need a helping hand?

--- Additional comment from poma on 2014-11-01 05:42:46 EDT ---


- Configuration file /usr/lib/systemd/system/wpa_supplicant.service is marked executable.
  https://bugzilla.redhat.com/show_bug.cgi?id=1159084

- No need to set the service(configuration) file as an executable.
  https://bugzilla.redhat.com/show_bug.cgi?id=1157200

- Execute permission on wpa_supplicant.service unit file
  https://bugzilla.redhat.com/show_bug.cgi?id=1146014

- please remove +x bit from systemd unit file
  https://bugzilla.redhat.com/show_bug.cgi?id=963915

--- Additional comment from Michael Catanzaro on 2014-11-01 09:40:45 EDT ---



--- Additional comment from Michael Catanzaro on 2014-11-01 09:40:50 EDT ---



--- Additional comment from Michael Catanzaro on 2014-11-01 09:41:02 EDT ---



--- Additional comment from Michael Catanzaro on 2014-11-01 09:41:10 EDT ---



--- Additional comment from Dan Williams on 2015-02-26 15:26:39 EST ---



--- Additional comment from Gabriel Somlo on 2015-03-25 20:03:31 EDT ---

Any chance this could be built and pushed for F21 as well ?

I'm getting the same problem on my up-to-date F21 with wpa_supplicant-2.0.12

Thanks!

Comment 3 Mario Trangoni 2016-03-03 13:34:08 UTC

The sources are OK. wpa_supplicant.service has the correct permissions there.

But in system we can see: 

root@host ~ # ll /lib/systemd/system/wpa_supplicant.service
-rwxr-xr-x. 1 root root 365 Jun 15  2015 /lib/systemd/system/wpa_supplicant.service*
root@host ~ # rpm -qf /lib/systemd/system/wpa_supplicant.service
wpa_supplicant-2.0-17.el7_1.x86_64

It should be 0664 and not 0755.

Comment 4 Mario Trangoni 2016-03-03 14:19:10 UTC

I forgot to write why I (re)open this bug. With 'journalctl -b' we can see:

Feb 09 17:04:02 host dbus[1381]: [system] Activating via systemd: service name='fi.w1.wpa_supplicant1' unit='wpa_supplicant.service'
Feb 09 17:04:02 host dbus-daemon[1381]: dbus[1381]: [system] Activating via systemd: service name='fi.w1.wpa_supplicant1' unit='wpa_supplicant.service'
Feb 09 17:04:02 host systemd[1]: Configuration file /usr/lib/systemd/system/wpa_supplicant.service is marked executable. Please remove executable permission bits. Proceeding anyway.
Feb 09 17:04:02 host systemd[1]: Starting WPA Supplicant daemon...
Feb 09 17:04:02 host systemd[1]: Starting Authorization Manager...
Feb 09 17:04:02 host dbus[1381]: [system] Successfully activated service 'fi.w1.wpa_supplicant1'
Feb 09 17:04:02 host dbus-daemon[1381]: dbus[1381]: [system] Successfully activated service 'fi.w1.wpa_supplicant1'
Feb 09 17:04:02 host NetworkManager[1853]: <info>  wpa_supplicant running
Feb 09 17:04:02 host systemd[1]: Started WPA Supplicant daemon.

This is why we want to have this problem fixed.

Comment 5 Mario Trangoni 2016-03-03 15:30:57 UTC

At line 131 of 'wpa_supplicant.spec'

# init scripts
install -D -m 0755 %{SOURCE3} %{buildroot}/%{_unitdir}/%{name}.service

and it has to be:

# init scripts
install -D -m 0644 %{SOURCE3} %{buildroot}/%{_unitdir}/%{name}.service

Comment 6 Lubomir Rintel 2016-03-17 09:09:57 UTC


*** This bug has been marked as a duplicate of bug 1286965 ***