1
0
mirror of https://github.com/samba-team/samba.git synced 2024-12-23 17:34:34 +03:00

libreplace: Add a closefrom() implementation

There is closefrom in some BSDs, but Linux ships this only as part
of libbsd.  Add a new implementation of it in libreplace. The one in
libbsd of jessie and upstream differ and it has for example optimizations
for FreeBSD, but it gets some of the array calculations slightly wrong
from my point of view. If you want those, use libbsd. This replacement
is optimized on Linux only looking at /proc/self/fd/, everything else
would do the OPEN_MAX brute force fallback.

Signed-off-by: Volker Lendecke <vl@samba.org>
This commit is contained in:
Volker Lendecke 2016-06-09 21:27:09 +02:00 committed by Andreas Schneider
parent 467ea855cc
commit 55529d0f85
3 changed files with 150 additions and 0 deletions

138
lib/replace/closefrom.c Normal file
View File

@ -0,0 +1,138 @@
/*
* Unix SMB/CIFS implementation.
* Samba utility functions
* Copyright (C) Volker Lendecke 2016
*
* ** NOTE! The following LGPL license applies to the replace
* ** library. This does NOT imply that all of Samba is released
* ** under the LGPL
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
* License as published by the Free Software Foundation; either
* version 3 of the License, or (at your option) any later version.
*
* This library is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* Library General Public License for more details.
*
* You should have received a copy of the GNU Lesser General Public
* License along with this library; if not, see <http://www.gnu.org/licenses/>.
*/
#include "replace.h"
#include <dirent.h>
#include <unistd.h>
#include <limits.h>
static int closefrom_sysconf(int lower)
{
long max_files, fd;
max_files = sysconf(_SC_OPEN_MAX);
if (max_files == -1) {
max_files = 65536;
}
for (fd=lower; fd<max_files; fd++) {
close(fd);
}
return 0;
}
static int closefrom_procfs(int lower)
{
DIR *dirp;
int dir_fd;
struct dirent *dp;
int *fds = NULL;
size_t num_fds = 0;
size_t fd_array_size = 0;
size_t i;
int ret = ENOMEM;
dirp = opendir("/proc/self/fd");
if (dirp == 0) {
return errno;
}
dir_fd = dirfd(dirp);
if (dir_fd == -1) {
ret = errno;
goto fail;
}
while ((dp = readdir(dirp)) != NULL) {
char *endptr;
unsigned long long fd;
errno = 0;
fd = strtoull(dp->d_name, &endptr, 10);
if ((fd == 0) && (errno == EINVAL)) {
continue;
}
if ((fd == ULLONG_MAX) && (errno == ERANGE)) {
continue;
}
if (*endptr != '\0') {
continue;
}
if (fd == dir_fd) {
continue;
}
if (fd > INT_MAX) {
continue;
}
if (fd < lower) {
continue;
}
if (num_fds >= (fd_array_size / sizeof(int))) {
void *tmp;
if (fd_array_size == 0) {
fd_array_size = 16 * sizeof(int);
} else {
if (fd_array_size + fd_array_size <
fd_array_size) {
/* overflow */
goto fail;
}
fd_array_size = fd_array_size + fd_array_size;
}
tmp = realloc(fds, fd_array_size);
if (tmp == NULL) {
goto fail;
}
fds = tmp;
}
fds[num_fds++] = fd;
}
for (i=0; i<num_fds; i++) {
close(fds[i]);
}
ret = 0;
fail:
closedir(dirp);
free(fds);
return ret;
}
int rep_closefrom(int lower)
{
int ret;
ret = closefrom_procfs(lower);
if (ret == 0) {
return 0;
}
return closefrom_sysconf(lower);
}

View File

@ -247,6 +247,12 @@ size_t rep_strlcpy(char *d, const char *s, size_t bufsize);
size_t rep_strlcat(char *d, const char *s, size_t bufsize);
#endif
#ifndef HAVE_CLOSEFROM
#define closefrom rep_closefrom
int rep_closefrom(int lower);
#endif
#if (defined(BROKEN_STRNDUP) || !defined(HAVE_STRNDUP))
#undef HAVE_STRNDUP
#define strndup rep_strndup

View File

@ -257,6 +257,9 @@ def configure(conf):
if not conf.CHECK_FUNCS_IN('setproctitle', 'setproctitle', headers='setproctitle.h'):
conf.CHECK_FUNCS_IN('setproctitle', 'bsd', headers='sys/types.h bsd/unistd.h')
if not conf.CHECK_FUNCS('closefrom'):
conf.CHECK_FUNCS_IN('closefrom', 'bsd', headers='bsd/unistd.h')
conf.CHECK_CODE('''
struct ucred cred;
socklen_t cred_len;
@ -683,6 +686,9 @@ def build(bld):
if not bld.CONFIG_SET('HAVE_GETXATTR') or bld.CONFIG_SET('XATTR_ADDITIONAL_OPTIONS'):
REPLACE_SOURCE += ' xattr.c'
if not bld.CONFIG_SET('HAVE_CLOSEFROM'):
REPLACE_SOURCE += ' closefrom.c'
bld.SAMBA_LIBRARY('replace',
source=REPLACE_SOURCE,
group='base_libraries',