605754 – net-analyzer/suricata init script

Bug 605754 - net-analyzer/suricata init script

Summary: net-analyzer/suricata init script

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Sławek Lis (RETIRED)

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2017-01-15 11:05 UTC by Vieri
Modified:	2017-01-16 12:28 UTC (History)
CC List:	2 users (show)

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

Attachments
patch for files/suricata-3.2-init (suricata-3.2-init.diff,398 bytes, patch) 2017-01-15 11:05 UTC, Vieri	Details \| Diff
patch for the init script (suricata-3.2-init.diff,2.00 KB, patch) 2017-01-16 11:59 UTC, Vieri	Details \| Diff
patch for conf.d file (suricata-3.2-conf.diff,554 bytes, patch) 2017-01-16 12:00 UTC, Vieri	Details \| Diff
Show Obsolete (1) 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 Vieri 2017-01-15 11:05:39 UTC

Created attachment 460132 [details, diff]
patch for files/suricata-3.2-init

Currently, if you run
/etc/init.d/suricata restart
and the suricata process didn't really stop then start() will be called anyway because eend is always 0.

Some suricata processes can take a very long time to stop.

Attaching patch.

Comment 1 Sławek Lis (RETIRED) gentoo-dev

2017-01-16 10:33:41 UTC

I think it would be better to use start-stop-daemon --retry option, instead of repetitively checking in while loop.

Comment 2 Vieri 2017-01-16 11:59:38 UTC

Created attachment 460310 [details, diff]
patch for the init script

Thanks for the tip!

Comment 3 Vieri 2017-01-16 12:00:29 UTC

Created attachment 460312 [details, diff]
patch for conf.d file

Allows user to set retry timeout in conf.d.

Comment 4 Sławek Lis (RETIRED) gentoo-dev

2017-01-16 12:28:20 UTC

Updated