OpenVZ Forum: Devel » vzctl: race condition at open("/sbin/init")

Home » Mailing lists » Devel » vzctl: race condition at open("/sbin/init")

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

Re: vzctl: race condition at open("/sbin/init") [message #47917 is a reply to message #47277]

Tue, 18 September 2012 15:09

kir
Messages: 1645
Registered: August 2005
Location: Moscow, Russia

Senior Member

On 07/25/2012 11:07 PM, Vasily Kulikov wrote:
> Hi,
>
> stat()+open() is not atomic in the code below, so there is a race
> condition. A container root may change /sbin/init between these calls
> to e.g. FIFO and then make the vzctl's process hang up on read().
>
> I'd add O_NOCTTY to open's flags and change stat() before open() to
> fstat() just after open().

Thanks a lot for reporting!

Does this patch seems sufficient to you?
http://git.openvz.org/?p=vzctl;a=commitdiff;h=7c47a7953

>
>
> vzctl-3.3/src/lib/readelf.c:
>
> int get_arch_from_elf(const char *file)
> {
> ...
> if (stat(file, &st)) <<<<<
> return -1;
> if (!S_ISREG(st.st_mode))
> return -1;
> fd = open(file, O_RDONLY); <<<<<
> if (fd < 0)
> return -1;
> nbytes = read(fd, (void *) &elf_hdr, sizeof(elf_hdr));
> ...
> }
>
> Thanks,
>

Kir Kolyshkin

http://static.openvz.org/userbars/openvz-developer.png

Report message to a moderator

[Message index]

		vzctl: race condition at open("/sbin/init") By: Vasily Kulikov on Wed, 25 July 2012 19:07
		Re: vzctl: race condition at open("/sbin/init") By: kir on Tue, 18 September 2012 15:09
		Re: vzctl: race condition at open("/sbin/init") By: Vasily Kulikov on Tue, 18 September 2012 20:12

Previous Topic:	[VZCTL PATCH] dists: add distribution config file for Alpine Linux
Next Topic:	[PATCH v5 00/10] IPC: checkpoint/restore in userspace enhancements

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Sat Oct 25 18:03:31 GMT 2025

Total time taken to generate the page: 0.09281 seconds