Try shell in posix_spawn* only in compat mode

This commit is contained in:
Ulrich Drepper 2011-09-05 20:24:50 -04:00
parent 54b1f8b6bf
commit d96de9634a
8 changed files with 83 additions and 24 deletions

View File

@ -1,5 +1,20 @@
2011-09-05 Ulrich Drepper <drepper@gmail.com>
[BZ #13134]
* sysdeps/posix/spawni.c (script_execute): Define only for compatibility
before glibc 2.15.
(tryshell): Define.
(__spawni): Change last parameter to be flag. Test
SPAWN_XFLAGS_USE_PATH flag to use path or not.
Don't try to use shell unless SPAWN_XFLAGS_TRY_SHELL is set.
* sysdeps/mach/hurd/spawni.c: Change last parameter and adjust user.
* posix/spawni.c: Likewise.
* posix/spawn.c: Add compat version which passed SPAWN_XFLAGS_TRY_SHELL.
* posix/spawnp.c: Likewise. Change normal version to use
SPAWN_XFLAGS_USE_PATH.
* posix/spawn_int.c: Define SPAWN_XFLAGS_USE_PATH and
SPAWN_XFLAGS_TRY_SHELL.
[BZ #13150]
* posix/glob.h: Remove gcc 1.x support.

2
NEWS
View File

@ -10,7 +10,7 @@ Version 2.15
* The following bugs are resolved with this release:
9696, 12868, 12852, 12874, 12885, 12907, 12922, 12935, 13007, 13021,
13068, 13114, 13144, 13150
13068, 13114, 13134, 13144, 13150
* New program pldd to list loaded object of a process
Implemented by Ulrich Drepper.

View File

@ -1,4 +1,4 @@
/* Copyright (C) 2000 Free Software Foundation, Inc.
/* Copyright (C) 2000, 2011 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
@ -18,14 +18,31 @@
#include <spawn.h>
#include "spawn_int.h"
#include <shlib-compat.h>
/* Spawn a new process executing PATH with the attributes describes in *ATTRP.
Before running the process perform the actions described in FILE-ACTIONS. */
int
posix_spawn (pid_t *pid, const char *path,
const posix_spawn_file_actions_t *file_actions,
const posix_spawnattr_t *attrp, char *const argv[],
char *const envp[])
__posix_spawn (pid_t *pid, const char *path,
const posix_spawn_file_actions_t *file_actions,
const posix_spawnattr_t *attrp, char *const argv[],
char *const envp[])
{
return __spawni (pid, path, file_actions, attrp, argv, envp, 0);
}
versioned_symbol (libc, __posix_spawn, posix_spawn, GLIBC_2_15);
#if SHLIB_COMPAT (libc, GLIBC_2_2, GLIBC_2_15)
int
attribute_compat_text_section
__posix_spawn_compat (pid_t *pid, const char *file,
const posix_spawn_file_actions_t *file_actions,
const posix_spawnattr_t *attrp, char *const argv[],
char *const envp[])
{
return __spawni (pid, file, file_actions, attrp, argv, envp,
SPAWN_XFLAGS_TRY_SHELL);
}
compat_symbol (libc, __posix_spawn_compat, posix_spawn, GLIBC_2_2);
#endif

View File

@ -29,10 +29,13 @@ struct __spawn_action
} action;
};
#define SPAWN_XFLAGS_USE_PATH 0x1
#define SPAWN_XFLAGS_TRY_SHELL 0x2
extern int __posix_spawn_file_actions_realloc (posix_spawn_file_actions_t *
file_actions);
extern int __spawni (pid_t *pid, const char *path,
const posix_spawn_file_actions_t *file_actions,
const posix_spawnattr_t *attrp, char *const argv[],
char *const envp[], int use_path);
char *const envp[], int xflags);

View File

@ -1,5 +1,5 @@
/* Guts of POSIX spawn interface. Stub version.
Copyright (C) 2001 Free Software Foundation, Inc.
Copyright (C) 2001, 2011 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
@ -36,7 +36,7 @@ int
__spawni (pid_t *pid, const char *file,
const posix_spawn_file_actions_t *file_actions,
const posix_spawnattr_t *attrp, char *const argv[],
char *const envp[], int use_path)
char *const envp[], int xflags)
{
__set_errno (ENOSYS);
return -1;

View File

@ -1,4 +1,4 @@
/* Copyright (C) 2000 Free Software Foundation, Inc.
/* Copyright (C) 2000, 2011 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
@ -18,14 +18,32 @@
#include <spawn.h>
#include "spawn_int.h"
#include <shlib-compat.h>
/* Spawn a new process executing FILE with the attributes describes in *ATTRP.
Before running the process perform the actions described in FILE-ACTIONS. */
int
posix_spawnp (pid_t *pid, const char *file,
const posix_spawn_file_actions_t *file_actions,
const posix_spawnattr_t *attrp, char *const argv[],
char *const envp[])
__posix_spawnp (pid_t *pid, const char *file,
const posix_spawn_file_actions_t *file_actions,
const posix_spawnattr_t *attrp, char *const argv[],
char *const envp[])
{
return __spawni (pid, file, file_actions, attrp, argv, envp, 1);
return __spawni (pid, file, file_actions, attrp, argv, envp,
SPAWN_XFLAGS_USE_PATH);
}
versioned_symbol (libc, __posix_spawnp, posix_spawnp, GLIBC_2_15);
#if SHLIB_COMPAT (libc, GLIBC_2_2, GLIBC_2_15)
int
attribute_compat_text_section
__posix_spawnp_compat (pid_t *pid, const char *file,
const posix_spawn_file_actions_t *file_actions,
const posix_spawnattr_t *attrp, char *const argv[],
char *const envp[])
{
return __spawni (pid, file, file_actions, attrp, argv, envp,
SPAWN_XFLAGS_USE_PATH | SPAWN_XFLAGS_TRY_SHELL);
}
compat_symbol (libc, __posix_spawnp_compat, posix_spawnp, GLIBC_2_2);
#endif

View File

@ -1,5 +1,5 @@
/* spawn a new process running an executable. Hurd version.
Copyright (C) 2001,02,04 Free Software Foundation, Inc.
Copyright (C) 2001,02,04,11 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
@ -41,7 +41,7 @@ __spawni (pid_t *pid, const char *file,
const posix_spawn_file_actions_t *file_actions,
const posix_spawnattr_t *attrp,
char *const argv[], char *const envp[],
int use_path)
int xflags)
{
pid_t new_pid;
char *path, *p, *name;
@ -391,7 +391,7 @@ __spawni (pid_t *pid, const char *file,
/* Make sure the dtable can hold NEWFD. */
#define EXPAND_DTABLE(newfd) \
({ \
({ \
if ((unsigned int)newfd >= dtablesize \
&& newfd < _hurd_rlimits[RLIMIT_OFILE].rlim_cur) \
{ \
@ -399,7 +399,7 @@ __spawni (pid_t *pid, const char *file,
NEW_TABLE (dtable, newfd); \
NEW_TABLE (ulink_dtable, newfd); \
NEW_TABLE (dtable_cells, newfd); \
dtablesize = newfd + 1; \
dtablesize = newfd + 1; \
} \
((unsigned int)newfd < dtablesize ? 0 : EMFILE); \
})
@ -543,7 +543,7 @@ __spawni (pid_t *pid, const char *file,
conditions are diagnosed first and what side effects (file creation,
etc) can be observed before what errors. */
if (! use_path || strchr (file, '/') != NULL)
if ((xflags & SPAWN_XFLAGS_USE_PATH) == 0 || strchr (file, '/') != NULL)
/* The FILE parameter is actually a path. */
err = child_lookup (file, O_EXEC, 0, &execfile);
else

View File

@ -28,6 +28,7 @@
#include "spawn_int.h"
#include <not-cancel.h>
#include <local-setxid.h>
#include <shlib-compat.h>
/* The Unix standard contains a long explanation of the way to signal
@ -38,6 +39,7 @@
#define SPAWN_ERROR 127
#if SHLIB_COMPAT (libc, GLIBC_2_2, GLIBC_2_15)
/* The file is accessible but it is not an executable file. Invoke
the shell to interpret it as a script. */
static void
@ -64,6 +66,10 @@ script_execute (const char *file, char *const argv[], char *const envp[])
__execve (new_argv[0], new_argv, envp);
}
}
# define tryshell (xflags & SPAWN_XFLAGS_TRY_SHELL)
#else
# define tryshell 0
#endif
/* Spawn a new process executing PATH with the attributes describes in *ATTRP.
@ -72,7 +78,7 @@ int
__spawni (pid_t *pid, const char *file,
const posix_spawn_file_actions_t *file_actions,
const posix_spawnattr_t *attrp, char *const argv[],
char *const envp[], int use_path)
char *const envp[], int xflags)
{
pid_t new_pid;
char *path, *p, *name;
@ -226,12 +232,12 @@ __spawni (pid_t *pid, const char *file,
}
}
if (! use_path || strchr (file, '/') != NULL)
if ((xflags & SPAWN_XFLAGS_USE_PATH) == 0 || strchr (file, '/') != NULL)
{
/* The FILE parameter is actually a path. */
__execve (file, argv, envp);
if (errno == ENOEXEC)
if (tryshell && errno == ENOEXEC)
script_execute (file, argv, envp);
/* Oh, oh. `execve' returns. This is bad. */
@ -277,7 +283,7 @@ __spawni (pid_t *pid, const char *file,
/* Try to execute this name. If it works, execv will not return. */
__execve (startp, argv, envp);
if (errno == ENOEXEC)
if (tryshell && errno == ENOEXEC)
script_execute (startp, argv, envp);
switch (errno)