2001-05-19 Roland McGrath <roland@frob.com>

* hurd/hurdlookup.c (__hurd_file_name_lookup_retry): Don't leak the
	result port when failing with ELOOP.  Do the ELOOP check when
	following an absolute symlink target as well as relative.
	Remove #ifdef SYMLOOP_MAX conditional--it should always be defined.
This commit is contained in:
Roland McGrath 2001-05-20 02:27:44 +00:00
parent 0e17ee3c71
commit a500fbcd53
1 changed files with 7 additions and 4 deletions

View File

@ -1,4 +1,4 @@
/* Copyright (C) 1992, 93, 94, 95, 96, 97, 99 Free Software Foundation, Inc.
/* Copyright (C) 1992,93,94,95,96,97,99,2001 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
@ -164,10 +164,11 @@ __hurd_file_name_lookup_retry (error_t (*use_init_port)
/* Fall through. */
case FS_RETRY_NORMAL:
#ifdef SYMLOOP_MAX
if (nloops++ >= SYMLOOP_MAX)
return ELOOP;
#endif
{
__mach_port_deallocate (__mach_task_self (), *result);
return ELOOP;
}
/* An empty RETRYNAME indicates we have the final port. */
if (retryname[0] == '\0' &&
@ -240,6 +241,8 @@ __hurd_file_name_lookup_retry (error_t (*use_init_port)
dirport = INIT_PORT_CRDIR;
if (*result != MACH_PORT_NULL)
__mach_port_deallocate (__mach_task_self (), *result);
if (nloops++ >= SYMLOOP_MAX)
return ELOOP;
file_name = &retryname[1];
break;