[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sat, 8 Feb 2020 10:45:10 +0300
From: Alexander Scherbatiy <alexander.scherbatiy@...l-sw.com>
To: musl@...ts.openwall.com
Subject: Re: fopen with "e" mode to close file descriptor in exec... functions
Below are steps to reproduce it in docker, logs from docker and strace log.
> docker run --rm -it alpine:3.11.3 ash
> apk add gcc
> apk add libc-dev
Copy the posix_spawn_sample.c code below (note it uses "ash" in 'char
*argv[] = {"ash", ,,,}' for posix_spawn on Alpine Linux )
> gcc -o posix_spawn_sample posix_spawn_sample.c
> ./posix_spawn_sample
--- output ---
/ # ./posix_spawn_sample
Child pid: 17
PID=17
1 /bin/busybox /dev/pts/0
1 /bin/busybox /dev/pts/0
1 /bin/busybox /dev/pts/0
1 /bin/busybox /dev/tty
16 /posix_spawn_sample /dev/pts/0
16 /posix_spawn_sample /dev/pts/0
16 /posix_spawn_sample /dev/pts/0
16 /posix_spawn_sample /test.log
----------------
Note that "test.log" file is listed by "lsof -p PID" command.
---------- posix_spawn_sample.c ----------
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <unistd.h>
#include <spawn.h>
#include <sys/wait.h>
extern char **environ;
int main()
{
FILE* file = fopen("test.log", "ae");
fprintf(file, "test line\n");
pid_t pid;
// use "ash" for Alpine Linux and "bash" for Ubuntu
char *argv[] = {"ash", "-c", "echo PID=$$ && lsof -p $$", NULL};
int status = posix_spawn(&pid, "/bin/sh", NULL, NULL, argv, environ);
printf("Child pid: %i\n", pid);
waitpid(pid, &status, 0);
fclose(file);
return 0;
}
--------------------------
The full output step by step:
--------------------------------------
> docker run --rm -it alpine:3.11.3 ash
/ # apk add gcc
fetch http://dl-cdn.alpinelinux.org/alpine/v3.11/main/x86_64/APKINDEX.tar.gz
fetch
http://dl-cdn.alpinelinux.org/alpine/v3.11/community/x86_64/APKINDEX.tar.gz
(1/10) Installing libgcc (9.2.0-r3)
(2/10) Installing libstdc++ (9.2.0-r3)
(3/10) Installing binutils (2.33.1-r0)
(4/10) Installing gmp (6.1.2-r1)
(5/10) Installing isl (0.18-r0)
(6/10) Installing libgomp (9.2.0-r3)
(7/10) Installing libatomic (9.2.0-r3)
(8/10) Installing mpfr4 (4.0.2-r1)
(9/10) Installing mpc1 (1.1.0-r1)
(10/10) Installing gcc (9.2.0-r3)
Executing busybox-1.31.1-r9.trigger
OK: 101 MiB in 24 packages
/ # apk add libc-dev
(1/2) Installing musl-dev (1.1.24-r0)
(2/2) Installing libc-dev (0.7.2-r0)
OK: 111 MiB in 26 packages
/ # gcc -o posix_spawn_sample posix_spawn_sample.c
/ # ./posix_spawn_sample
Child pid: 17
PID=17
1 /bin/busybox /dev/pts/0
1 /bin/busybox /dev/pts/0
1 /bin/busybox /dev/pts/0
1 /bin/busybox /dev/tty
16 /posix_spawn_sample /dev/pts/0
16 /posix_spawn_sample /dev/pts/0
16 /posix_spawn_sample /dev/pts/0
16 /posix_spawn_sample /test.log
--------------------------------------
strace log
--------------------------------------
/ # strace -f ./posix_spawn_sample
execve("./posix_spawn_sample", ["./posix_spawn_sample"], 0x7ffd951eb318 /*
6 vars */) = 0
arch_prctl(ARCH_SET_FS, 0x7f0fa33d6d48) = 0
set_tid_address(0x7f0fa33d731c) = 31
mprotect(0x7f0fa33d3000, 4096, PROT_READ) = 0
mprotect(0x55e26cf7b000, 4096, PROT_READ) = 0
open("test.log", O_WRONLY|O_CREAT|O_APPEND|O_CLOEXEC, 0666) = 3
fcntl(3, F_SETFD, FD_CLOEXEC) = 0
fcntl(3, F_SETFD, FD_CLOEXEC) = 0
fcntl(3, F_GETFL) = 0x8401 (flags
O_WRONLY|O_APPEND|O_LARGEFILE)
ioctl(3, TIOCGWINSZ, 0x7ffd87e838a8) = -1 ENOTTY (Not a tty)
pipe2([4, 5], O_CLOEXEC) = 0
rt_sigprocmask(SIG_BLOCK, ~[], [], 8) = 0
clone(child_stack=0x7ffd87e838b8, flags=CLONE_VM|CLONE_VFORK|SIGCHLDstrace:
Process 32 attached
<unfinished ...>
[pid 32] close(4) = 0
[pid 32] fcntl(5, F_SETFD, FD_CLOEXEC) = 0
[pid 32] rt_sigprocmask(SIG_SETMASK, [], NULL, 8) = 0
[pid 32] execve("/bin/sh", ["ash", "-c", "echo PID=$$ && lsof -p $$"],
0x7ffd87e83998 /* 6 vars */ <unfinished ...>
[pid 31] <... clone resumed>) = 32
[pid 32] <... execve resumed>) = 0
[pid 31] close(5 <unfinished ...>
[pid 32] arch_prctl(ARCH_SET_FS, 0x7f1a19d9ed48 <unfinished ...>
[pid 31] <... close resumed>) = 0
[pid 31] read(4, <unfinished ...>
[pid 32] <... arch_prctl resumed>) = 0
[pid 31] <... read resumed>"", 4) = 0
[pid 32] set_tid_address(0x7f1a19d9f31c <unfinished ...>
[pid 31] close(4 <unfinished ...>
[pid 32] <... set_tid_address resumed>) = 32
[pid 31] <... close resumed>) = 0
[pid 32] mprotect(0x7f1a19d9b000, 4096, PROT_READ <unfinished ...>
[pid 31] rt_sigprocmask(SIG_SETMASK, [], NULL, 8) = 0
[pid 32] <... mprotect resumed>) = 0
[pid 31] ioctl(1, TIOCGWINSZ, {ws_row=40, ws_col=155, ws_xpixel=0,
ws_ypixel=0}) = 0
[pid 32] mprotect(0x556eae13a000, 16384, PROT_READ <unfinished ...>
[pid 31] writev(1, [{iov_base="Child pid: 32", iov_len=13},
{iov_base="\n", iov_len=1}], 2 <unfinished ...>
[pid 32] <... mprotect resumed>) = 0Child pid: 32
[pid 31] <... writev resumed>) = 14
[pid 31] wait4(32, <unfinished ...>
[pid 32] getuid() = 0
[pid 32] getpid() = 32
[pid 32] rt_sigprocmask(SIG_UNBLOCK, [RT_1 RT_2], NULL, 8) = 0
[pid 32] rt_sigaction(SIGCHLD, {sa_handler=0x556eae0b9dfc,
sa_mask=~[RTMIN RT_1 RT_2], sa_flags=SA_RESTORER,
sa_restorer=0x7f1a19d500d4}, NULL, 8) = 0
[pid 32] rt_sigaction(SIGHUP, {sa_handler=SIG_DFL, sa_mask=[],
sa_flags=SA_RESTORER|SA_RESTART, sa_restorer=0x7f1a19d500d4},
{sa_handler=SIG_DFL, sa_mask=[], sa_flags=0}, 8) = 0
[pid 32] getppid() = 31
[pid 32] stat("/", {st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
[pid 32] stat(".", {st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
[pid 32] rt_sigaction(SIGINT, NULL, {sa_handler=SIG_DFL, sa_mask=[],
sa_flags=0}, 8) = 0
[pid 32] rt_sigaction(SIGINT, {sa_handler=0x556eae0b9dfc,
sa_mask=~[RTMIN RT_1 RT_2], sa_flags=SA_RESTORER,
sa_restorer=0x7f1a19d500d4}, NULL, 8) = 0
[pid 32] rt_sigaction(SIGQUIT, NULL, {sa_handler=SIG_DFL, sa_mask=[],
sa_flags=0}, 8) = 0
[pid 32] rt_sigaction(SIGQUIT, {sa_handler=SIG_IGN, sa_mask=~[RTMIN RT_1
RT_2], sa_flags=SA_RESTORER, sa_restorer=0x7f1a19d500d4}, NULL, 8) = 0
[pid 32] rt_sigaction(SIGTERM, NULL, {sa_handler=SIG_DFL, sa_mask=[],
sa_flags=0}, 8) = 0
[pid 32] wait4(-1, 0x7fff4e3ea22c, WNOHANG, NULL) = -1 ECHILD (No child
process)
[pid 32] write(1, "PID=32\n", 7PID=32
) = 7
[pid 32] stat("/usr/local/sbin/lsof", 0x7fff4e3ea170) = -1 ENOENT (No
such file or directory)
[pid 32] stat("/usr/local/bin/lsof", 0x7fff4e3ea170) = -1 ENOENT (No
such file or directory)
[pid 32] stat("/usr/sbin/lsof", 0x7fff4e3ea170) = -1 ENOENT (No such
file or directory)
[pid 32] stat("/usr/bin/lsof", {st_mode=S_IFREG|0755, st_size=841288,
...}) = 0
[pid 32] brk(NULL) = 0x556eaf7c5000
[pid 32] brk(0x556eaf7c6000) = 0x556eaf7c6000
[pid 32] execve("/usr/bin/lsof", ["lsof", "-p", "32"], 0x556eae13ed80 /*
6 vars */) = 0
[pid 32] arch_prctl(ARCH_SET_FS, 0x7fe42bc41d48) = 0
[pid 32] set_tid_address(0x7fe42bc4231c) = 32
[pid 32] mprotect(0x7fe42bc3e000, 4096, PROT_READ) = 0
[pid 32] mprotect(0x55d56c79b000, 16384, PROT_READ) = 0
[pid 32] getuid() = 0
[pid 32] open("/proc", O_RDONLY|O_CLOEXEC|O_DIRECTORY) = 3
[pid 32] fcntl(3, F_SETFD, FD_CLOEXEC) = 0
[pid 32] getdents64(3, /* 67 entries */, 2048) = 1984
[pid 32] readlink("/proc/1/exe", "/bin/busybox", 80) = 12
[pid 32] getpid() = 32
[pid 32] open("/proc/1/fd/", O_RDONLY|O_CLOEXEC|O_DIRECTORY) = 4
[pid 32] fcntl(4, F_SETFD, FD_CLOEXEC) = 0
[pid 32] brk(NULL) = 0x55d56dac0000
[pid 32] brk(0x55d56dac1000) = 0x55d56dac1000
[pid 32] getdents64(4, /* 6 entries */, 2048) = 144
[pid 32] readlink("/proc/1/fd/0", "/dev/pts/0", 80) = 10
[pid 32] ioctl(1, TIOCGWINSZ, {ws_row=40, ws_col=155, ws_xpixel=0,
ws_ypixel=0}) = 0
[pid 32] writev(1, [{iov_base="1\t/bin/busybox\t/dev/pts/0",
iov_len=25}, {iov_base="\n", iov_len=1}], 21 /bin/busybox /dev/pts/0
) = 26
[pid 32] readlink("/proc/1/fd/1", "/dev/pts/0", 80) = 10
[pid 32] writev(1, [{iov_base="1\t/bin/busybox\t/dev/pts/0",
iov_len=25}, {iov_base="\n", iov_len=1}], 21 /bin/busybox /dev/pts/0
) = 26
[pid 32] readlink("/proc/1/fd/2", "/dev/pts/0", 80) = 10
[pid 32] writev(1, [{iov_base="1\t/bin/busybox\t/dev/pts/0",
iov_len=25}, {iov_base="\n", iov_len=1}], 21 /bin/busybox /dev/pts/0
) = 26
[pid 32] readlink("/proc/1/fd/10", "/dev/tty", 80) = 8
[pid 32] writev(1, [{iov_base="1\t/bin/busybox\t/dev/tty", iov_len=23},
{iov_base="\n", iov_len=1}], 21 /bin/busybox /dev/tty
) = 24
[pid 32] getdents64(4, /* 0 entries */, 2048) = 0
[pid 32] close(4) = 0
[pid 32] readlink("/proc/28/exe", "/usr/bin/strace", 80) = 15
[pid 32] getpid() = 32
[pid 32] open("/proc/28/fd/", O_RDONLY|O_CLOEXEC|O_DIRECTORY) = 4
[pid 32] fcntl(4, F_SETFD, FD_CLOEXEC) = 0
[pid 32] getdents64(4, /* 5 entries */, 2048) = 120
[pid 32] readlink("/proc/28/fd/0", "pipe:[89317]", 80) = 12
[pid 32] writev(1, [{iov_base="28\t/usr/bin/strace\tpipe:[89317]",
iov_len=31}, {iov_base="\n", iov_len=1}], 228 /usr/bin/strace pipe:[89317]
) = 32
[pid 32] readlink("/proc/28/fd/1", "pipe:[89318]", 80) = 12
[pid 32] writev(1, [{iov_base="28\t/usr/bin/strace\tpipe:[89318]",
iov_len=31}, {iov_base="\n", iov_len=1}], 228 /usr/bin/strace pipe:[89318]
) = 32
[pid 32] readlink("/proc/28/fd/2", "/dev/pts/0", 80) = 10
[pid 32] writev(1, [{iov_base="28\t/usr/bin/strace\t/dev/pts/0",
iov_len=29}, {iov_base="\n", iov_len=1}], 228 /usr/bin/strace /dev/pts/0
) = 30
[pid 32] getdents64(4, /* 0 entries */, 2048) = 0
[pid 32] close(4) = 0
[pid 32] readlink("/proc/31/exe", "/posix_spawn_sample", 80) = 19
[pid 32] getpid() = 32
[pid 32] open("/proc/31/fd/", O_RDONLY|O_CLOEXEC|O_DIRECTORY) = 4
[pid 32] fcntl(4, F_SETFD, FD_CLOEXEC) = 0
[pid 32] getdents64(4, /* 6 entries */, 2048) = 144
[pid 32] readlink("/proc/31/fd/0", "/dev/pts/0", 80) = 10
[pid 32] writev(1, [{iov_base="31\t/posix_spawn_sample\t/dev/pts/"...,
iov_len=33}, {iov_base="\n", iov_len=1}], 231 /posix_spawn_sample /dev/pts/0
) = 34
[pid 32] readlink("/proc/31/fd/1", "/dev/pts/0", 80) = 10
[pid 32] writev(1, [{iov_base="31\t/posix_spawn_sample\t/dev/pts/"...,
iov_len=33}, {iov_base="\n", iov_len=1}], 231 /posix_spawn_sample /dev/pts/0
) = 34
[pid 32] readlink("/proc/31/fd/2", "/dev/pts/0", 80) = 10
[pid 32] writev(1, [{iov_base="31\t/posix_spawn_sample\t/dev/pts/"...,
iov_len=33}, {iov_base="\n", iov_len=1}], 231 /posix_spawn_sample /dev/pts/0
) = 34
[pid 32] readlink("/proc/31/fd/3", "/test.log", 80) = 9
[pid 32] writev(1, [{iov_base="31\t/posix_spawn_sample\t/test.log",
iov_len=32}, {iov_base="\n", iov_len=1}], 231 /posix_spawn_sample /test.log
) = 33
[pid 32] getdents64(4, /* 0 entries */, 2048) = 0
[pid 32] close(4) = 0
[pid 32] readlink("/proc/32/exe", "/bin/busybox", 80) = 12
[pid 32] getpid() = 32
[pid 32] getdents64(3, /* 0 entries */, 2048) = 0
[pid 32] close(3) = 0
[pid 32] exit_group(0) = ?
[pid 32] +++ exited with 0 +++
<... wait4 resumed>[{WIFEXITED(s) && WEXITSTATUS(s) == 0}], 0, NULL) = 32
--- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=32, si_uid=0,
si_status=0, si_utime=0, si_stime=0} ---
writev(3, [{iov_base="test line\n", iov_len=10}, {iov_base=NULL,
iov_len=0}], 2) = 10
close(3) = 0
exit_group(0) = ?
+++ exited with 0 +++
--------------------------------------
Thanks,
Alexander.
On Sat, Feb 8, 2020 at 12:03 AM Rich Felker <dalias@...c.org> wrote:
> On Fri, Feb 07, 2020 at 09:22:01PM +0300, Alexander Scherbatiy wrote:
> > Hello,
> >
> > Below is a simple program without any errors handling that uses
> > fopen with "ae" mode to open a "test.log" file with append access
> > and call posix_spawn to create a child process and list all open files
> for
> > the child process.
> >
> > The program output shows that the child process has "test.log" file as
> open.
> > I used docker with Alpine Linux 3.11.3 (musl libc x86_64 1.1.24).
> > Is it an expected behaviour for Alpine Linux?
> >
> > The same program on my Ubuntu 19.10 (with "ash" changed to "bash") shows
> > that
> > the child process doesn't have "test.log" file as open.
> >
> > Thanks,
> > Alexander.
> >
> > ---------- posix_spawn_sample.c ----------
> > #include <stdlib.h>
> > #include <stdio.h>
> > #include <string.h>
> >
> > #include <unistd.h>
> > #include <spawn.h>
> > #include <sys/wait.h>
> >
> > extern char **environ;
> >
> > int main()
> > {
> > FILE* file = fopen("test.log", "ae");
> > fprintf(file, "test line\n");
> >
> > pid_t pid;
> > char *argv[] = {"bash", "-c", "echo PID=$$ && lsof -p $$", NULL};
> > int status = posix_spawn(&pid, "/bin/sh", NULL, NULL, argv, environ);
> > printf("Child pid: %i\n", pid);
> > waitpid(pid, &status, 0);
> >
> > fclose(file);
> > return 0;
> > }
> > --------------------------
>
> I can't reproduce this. I had to change /bin/sh to /bin/bash to be
> able to run the test at all, and the lsof I have (busybox) doesn't
> support -p, but I can't see any indication that the child has the log
> file open. I also tried changing it to run "ls -l /proc/$$/fd" in the
> child, and a fd 3 shows up there but prints an error because fd 3 is
> the directory fd for listing /proc/$$/fd and it's closed before trying
> to stat it -- apparently ash execs ls rather than running it as a
> child. Indeed adding "; echo" makes that go away.
>
> If you're still having the problem, could you attach a log of running
> your test under "strace -f"?
>
> Rich
>
Content of type "text/html" skipped
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
