Bug 10551 – ntp: init fails if /var/chroot/ntp doesn't exist

Bug 10551 - ntp: init fails if /var/chroot/ntp doesn't exist

Summary:

ntp: init fails if /var/chroot/ntp doesn't exist

Status:	CLOSED FIXED

Product:	Codex
Classification:	Unclassified
Component:	utils
Version:	stable-rc grimoire
Platform:	x86 Linux

Importance:	P2 blocker
Assigned To:	Grimoire Bug List

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2006-02-17 06:46 UTC by Juuso Alasuutari
Modified:	2007-03-31 02:18 UTC (History)
CC List:	0 users

See Also:

Flags:
sobukus: fixed_in_lesser_branch+ seth: integrate_to_stable‑rc_grimoire+

Attachments
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 Juuso Alasuutari 2006-02-17 06:46:19 UTC

The ntp init script has "-i /var/chroot/ntp" in ARGS. This causes init to fail if there is no such directory.

Comment 1 Seth Woolley 2006-07-08 17:43:40 UTC

This bug now exists in stable-rc, version 0.5

Comment 2 Thomas Orgis 2007-01-19 20:28:45 UTC

Well, if we indeed hardcode that into the init script, then the spell simply 
should create /var/chroot/ntp.
Hm, or does this dir need to contain something? Well, ntpd wants to write to 
the driftfile...

Could treeve enlighten us with his config for where this spell works?
I just checked that I encountered this with my home ntp server and just 
commented out the user and jaildir part. I am not paranoid on ntp on this 
little home server...

A hotfix for the spell is to change the opts line to
ARGS="-p $PIDFILE "
That way it does actually work! It may not be the most secure setup, but if 
the user wants chroot, we will have to setup it (or the spell) and configure 
the init script (we should add sysconfig option for chroot/jaildir and user).

Hm, actually, I will do a commit now that does this since the init script 
cannot possibly work as it is and we should provide at least something that 
can work.
Then one can re-add proper support for chroot as sysconfig option and/or the 
spell offering to create /var/chroot/ntp .

Comment 3 Thomas Orgis 2007-01-19 20:53:56 UTC

Actually, it's better to simply fix the spell by creating 
that /var/chroot/ntp:
74f37fef1dae9d27968d0944beb1f3a03580ea35

request for integration... and marking this bug as fixed

Comment 4 Jaka Kranjc 2007-01-20 06:48:33 UTC

integrated.

Comment 5 Jeremy Blosser 2007-03-31 01:18:35 UTC

closing fixed bugs