[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH Hurd] Fix invalid port deallocation in `symlink'
From: |
Ludovic Courtès |
Subject: |
[PATCH Hurd] Fix invalid port deallocation in `symlink' |
Date: |
Sat, 17 Mar 2012 15:19:38 +0100 |
User-agent: |
Gnus/5.110018 (No Gnus v0.18) Emacs/24.0.93 (gnu/linux) |
Hello,
Currently, running ‘ln -s foo bar’ on a read-only file system leads to
an invalid port deallocation.
This patch fixes that.
Thanks,
Ludo’.
2012-03-17 Ludovic Courtès <ludo@gnu.org>
* sysdeps/mach/hurd/symlink.c (__symlink): Deallocate NODE only
when it is non-null.
diff --git a/sysdeps/mach/hurd/symlink.c b/sysdeps/mach/hurd/symlink.c
index 857e236..4a4e61f 100644
--- a/sysdeps/mach/hurd/symlink.c
+++ b/sysdeps/mach/hurd/symlink.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1991, 92, 93, 94, 95, 96, 97 Free Software Foundation, Inc.
+/* Copyright (C) 1991, 92, 93, 94, 95, 96, 97, 12 Free Software Foundation,
Inc.
This file is part of the GNU C Library.
The GNU C Library is free software; you can redistribute it and/or
@@ -44,6 +44,7 @@ __symlink (from, to)
return -1;
/* Create a new, unlinked node in the target directory. */
+ node = MACH_PORT_NULL;
err = __dir_mkfile (dir, O_WRITE, 0777 & ~_hurd_umask, &node);
if (! err)
@@ -59,7 +60,9 @@ __symlink (from, to)
err = __dir_link (dir, node, name, 1);
__mach_port_deallocate (__mach_task_self (), dir);
- __mach_port_deallocate (__mach_task_self (), node);
+
+ if (node != MACH_PORT_NULL)
+ __mach_port_deallocate (__mach_task_self (), node);
if (err)
return __hurd_fail (err);
- [PATCH Hurd] Fix invalid port deallocation in `symlink',
Ludovic Courtès <=