From 87ef29ca94bb5b4bb6a0b8ef40d387470d4baac3 Mon Sep 17 00:00:00 2001 From: Roland McGrath Date: Tue, 24 Apr 2012 13:12:12 -0700 Subject: Hurd: fix symlink/symlinkat error case not to do an extraneous __mach_port_deallocate. --- ChangeLog | 7 +++++++ sysdeps/mach/hurd/symlink.c | 25 ++++++++++++++----------- sysdeps/mach/hurd/symlinkat.c | 26 ++++++++++++++------------ 3 files changed, 35 insertions(+), 23 deletions(-) diff --git a/ChangeLog b/ChangeLog index fc74310545..1c13c37d11 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2012-04-24 Roland McGrath + + * sysdeps/mach/hurd/symlink.c: Don't call __mach_port_deallocate on + NODE when __dir_mkfile failed. + * sysdeps/mach/hurd/symlinkat.c: Likewise. + Reported by Ludovic Courtès . + 2012-04-24 Andreas Jaeger * Makerules (common-clean): Also remove gen-as-const-headers diff --git a/sysdeps/mach/hurd/symlink.c b/sysdeps/mach/hurd/symlink.c index 8bca405139..0754b1893b 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-2012 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 @@ -46,19 +46,22 @@ __symlink (from, to) err = __dir_mkfile (dir, O_WRITE, 0777 & ~_hurd_umask, &node); if (! err) - /* Set the node's translator to make it a symlink. */ - err = __file_set_translator (node, - FS_TRANS_EXCL|FS_TRANS_SET, - FS_TRANS_EXCL|FS_TRANS_SET, 0, - buf, sizeof (_HURD_SYMLINK) + len, - MACH_PORT_NULL, MACH_MSG_TYPE_COPY_SEND); + { + /* Set the node's translator to make it a symlink. */ + err = __file_set_translator (node, + FS_TRANS_EXCL|FS_TRANS_SET, + FS_TRANS_EXCL|FS_TRANS_SET, 0, + buf, sizeof (_HURD_SYMLINK) + len, + MACH_PORT_NULL, MACH_MSG_TYPE_COPY_SEND); - if (! err) - /* Link the node, now a valid symlink, into the target directory. */ - err = __dir_link (dir, node, name, 1); + if (! err) + /* Link the node, now a valid symlink, into the target directory. */ + err = __dir_link (dir, node, name, 1); + + __mach_port_deallocate (__mach_task_self (), node); + } __mach_port_deallocate (__mach_task_self (), dir); - __mach_port_deallocate (__mach_task_self (), node); if (err) return __hurd_fail (err); diff --git a/sysdeps/mach/hurd/symlinkat.c b/sysdeps/mach/hurd/symlinkat.c index 0fa667d7cd..ff11ecdc16 100644 --- a/sysdeps/mach/hurd/symlinkat.c +++ b/sysdeps/mach/hurd/symlinkat.c @@ -1,6 +1,5 @@ /* Create a symbolic link named relative to an open directory. Hurd version. - Copyright (C) 1991,1992,1993,1994,1995,1996,1997,2006 - Free Software Foundation, Inc. + Copyright (C) 1991-2012 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 @@ -53,19 +52,22 @@ symlinkat (from, fd, to) err = __dir_mkfile (dir, O_WRITE, 0777 & ~_hurd_umask, &node); if (! err) - /* Set the node's translator to make it a symlink. */ - err = __file_set_translator (node, - FS_TRANS_EXCL|FS_TRANS_SET, - FS_TRANS_EXCL|FS_TRANS_SET, 0, - buf, sizeof (_HURD_SYMLINK) + len, - MACH_PORT_NULL, MACH_MSG_TYPE_COPY_SEND); + { + /* Set the node's translator to make it a symlink. */ + err = __file_set_translator (node, + FS_TRANS_EXCL|FS_TRANS_SET, + FS_TRANS_EXCL|FS_TRANS_SET, 0, + buf, sizeof (_HURD_SYMLINK) + len, + MACH_PORT_NULL, MACH_MSG_TYPE_COPY_SEND); - if (! err) - /* Link the node, now a valid symlink, into the target directory. */ - err = __dir_link (dir, node, name, 1); + if (! err) + /* Link the node, now a valid symlink, into the target directory. */ + err = __dir_link (dir, node, name, 1); + + __mach_port_deallocate (__mach_task_self (), node); + } __mach_port_deallocate (__mach_task_self (), dir); - __mach_port_deallocate (__mach_task_self (), node); if (err) return __hurd_fail (err); -- cgit 1.4.1