Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 452786 - net-analyzer/snort-2.9.2.3 init script snort.rc11 does not create /run/snort/
Summary: net-analyzer/snort-2.9.2.3 init script snort.rc11 does not create /run/snort/
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Patrick Lauer
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: keepdir-var-run_lock
  Show dependency tree
 
Reported: 2013-01-18 09:50 UTC by Peter Wilmott (RETIRED)
Modified: 2013-04-10 18:32 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
patch for snort.rc11 (snort.rc11.patch,305 bytes, text/plain)
2013-01-18 09:51 UTC, Peter Wilmott (RETIRED) 		Details
Use checkpath not mkdir (snort.rc11.patch,343 bytes, text/plain)
2013-01-21 07:24 UTC, Peter Wilmott (RETIRED) 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Wilmott (RETIRED) gentoo-dev 2013-01-18 09:50:42 UTC 
On systems with /var/run linked to /run, net-analyzer/snort-2.9.2.3 fails to start after a reboot due to the /var/run/snort directory being removed from tmpfs.

Reproducible: Always

Steps to Reproduce:
1.Install net-analyzer/snort
2.Link /var/run to /run (If not already done)
3.Reboot system
Actual Results:  
Snort fails to start with the following error:

/etc/init.d/snort[13204]: ERROR: snort failed to start



Patching the init script with the following check for /var/run/snort fixes the issue.
Comment 1 Peter Wilmott (RETIRED) gentoo-dev 2013-01-18 09:51:20 UTC 
Created attachment 336030 [details]
patch for snort.rc11
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2013-01-19 16:16:41 UTC 
It should of course stop moving /var/run/foo altogether.
Comment 3 Diego Elio Pettenò (RETIRED) gentoo-dev 2013-01-19 22:15:21 UTC 
Please use checkpath, not mkdir.
Comment 4 Peter Wilmott (RETIRED) gentoo-dev 2013-01-21 07:24:30 UTC 
Created attachment 336310 [details]
Use checkpath not mkdir

As requested.
Comment 5 Tony Vroon (RETIRED) gentoo-dev 2013-03-27 09:56:12 UTC 
Declaring intention to commit Peter Wilmott's second patch. If opposed, please state your objection.
Comment 6 Tony Vroon (RETIRED) gentoo-dev 2013-04-10 18:32:48 UTC 
+*snort-2.9.2.3-r1 (10 Apr 2013)
+
+  10 Apr 2013; Tony Vroon <chainsaw@gentoo.org> +snort-2.9.2.3-r1.ebuild,
+  +files/snort.rc12:
+  Init script fixes by Brian De Wolf to cease using deprecated opts variable,
+  closes bug #447780. /run tmpfs coping measures by Peter Wilmott close bug
+  #452786. Ebuild ported to EAPI 5, using REQUIRED_USE and dropping now
+  unnecessary die statements for ebuild helper functions.

Thank you for your contribution to Gentoo Linux Peter. Sorry for the delay, as I am not the maintainer there is some red tape for me to wade through.