Re: [PATCH] unshare: Use rcu_assign_pointer when setting sighand

Janak Desai wrote:
> 
> Oleg Nesterov wrote:
> >
> >Btw, copy_process() forbids CLONE_SIGHAND without CLONE_VM (is there a
> >good reason for that?), but one can do unshare(CLONE_VM). This is odd.
> >
> >
> Yes, copy_process forbids cloning of signal handlers without cloning of vm.
> However, it does allow cloning of vm without cloning of signal handlers. For
> those processes, that are sharing vm but not signal handlers, unsharing
> of vm
> is allowed.

Yes, I was wrong, I missed check_unshare_flags(),

	/*
	 * If unsharing vm, must also unshare signal handlers.
	 */
	if (*flags_ptr & CLONE_VM)
		*flags_ptr |= CLONE_SIGHAND;

Looking below,

	/*
	 * If unsharing signal handlers and the task was created
	 * using CLONE_THREAD, then must unshare the thread
	 */
	if ((*flags_ptr & CLONE_SIGHAND) &&
	    (atomic_read(&current->signal->count) > 1))
		*flags_ptr |= CLONE_THREAD;

Shouldn't this be:

		*flags_ptr |= (CLONE_THREAD | CLONE_VM)

? Currently it doesn't matter, but still.


However, I stronly beleive unshare(CLONE_VM) is buggy.

sys_unshare:


		if (new_mm) {
			...
			new_mm = mm;
		}

	...

	bad_unshare_cleanup_vm:
		if (new_mm)
			mmput(new_mm);


mmput() ignores mm->core_waiters.

No?

Oleg.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • [PATCH] for 2.6.16, disable unshare_vm()
    • From: Oleg Nesterov <[email protected]>

• References:
  • [PATCH] unshare: Cleanup up the sys_unshare interface before we are committed.
    • From: [email protected] (Eric W. Biederman)
  • [PATCH] unshare: Use rcu_assign_pointer when setting sighand
    • From: [email protected] (Eric W. Biederman)
  • Re: [PATCH] unshare: Use rcu_assign_pointer when setting sighand
    • From: Oleg Nesterov <[email protected]>
  • Re: [PATCH] unshare: Use rcu_assign_pointer when setting sighand
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH] unshare: Use rcu_assign_pointer when setting sighand
    • From: Oleg Nesterov <[email protected]>
  • Re: [PATCH] unshare: Use rcu_assign_pointer when setting sighand
    • From: Janak Desai <[email protected]>

• Prev by Date: [PATCH 04/18] RTC subsystem, MIPS cleanup
• Next by Date: [PATCH 18/18] RTC subsystem, M48T86 driver
• Previous by thread: Re: [PATCH] unshare: Use rcu_assign_pointer when setting sighand
• Next by thread: [PATCH] for 2.6.16, disable unshare_vm()
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]