The behaviour of printf()
seems to depend on the location of stdout
.
- If
stdout
is sent to the console, thenprintf()
is line-buffered and is flushed after a newline is printed. - If
stdout
is redirected to a file, the buffer is not flushed unlessfflush()
is called. - Moreover, if
printf()
is used beforestdout
is redirected to file, subsequent writes (to the file) are line-buffered and are flushed after newline.
When is stdout
line-buffered, and when does fflush()
need to be called?
Minimal example of each:
void RedirectStdout2File(const char* log_path) {
int fd = open(log_path, O_RDWR|O_APPEND|O_CREAT,S_IRWXU|S_IRWXG|S_IRWXO);
dup2(fd,STDOUT_FILENO);
if (fd != STDOUT_FILENO) close(fd);
}
int main_1(int argc, char* argv[]) {
/* Case 1: stdout is line-buffered when run from console */
printf("No redirect; printed immediately\n");
sleep(10);
}
int main_2a(int argc, char* argv[]) {
/* Case 2a: stdout is not line-buffered when redirected to file */
RedirectStdout2File(argv[0]);
printf("Will not go to file!\n");
RedirectStdout2File("/dev/null");
}
int main_2b(int argc, char* argv[]) {
/* Case 2b: flushing stdout does send output to file */
RedirectStdout2File(argv[0]);
printf("Will go to file if flushed\n");
fflush(stdout);
RedirectStdout2File("/dev/null");
}
int main_3(int argc, char* argv[]) {
/* Case 3: printf before redirect; printf is line-buffered after */
printf("Before redirect\n");
RedirectStdout2File(argv[0]);
printf("Does go to file!\n");
RedirectStdout2File("/dev/null");
}
Your are wrongly combining buffered and unbuffered IO functions. Such a combination must be done very carefully especially when the code has to be portable. (and it is bad to write unportable code...)
It is certainly best to avoid combining buffered and unbuffered IO on the same file descriptor.
Buffered IO:
fprintf()
,fopen()
,fclose()
,freopen()
...Unbuffered IO:
write()
,open()
,close()
,dup()
...When you use
dup2()
to redirect stdout. The function is not aware of the buffer which was filled byfprintf()
. So whendup2()
closes the old descriptor 1 it does not flush the buffer and the content could be flushed to a different output. In your case 2a it was sent to/dev/null
.The solution
In your case it is best to use
freopen()
instead ofdup2()
. This solves all your problems:FILE
stream. (case 2a)Here is the correct implementation of your function:
Unfortunately with buffered IO you cannot directly set permissions of a newly created file. You have to use other calls to change the permissions or you can use unportable glibc extensions. See the
fopen() man page
.Flushing for
stdout
is determined by its buffering behaviour. The buffering can be set to three modes:_IOFBF
(full buffering: waits untilfflush()
if possible),_IOLBF
(line buffering: newline triggers automatic flush), and_IONBF
(direct write always used). "Support for these characteristics is implementation-defined, and may be affected via thesetbuf()
andsetvbuf()
functions." [C99:7.19.3.3]"At program startup, three text streams are predefined and need not be opened explicitly — standard input (for reading conventional input), standard output (for writing conventional output), and standard error (for writing diagnostic output). As initially opened, the standard error stream is not fully buffered; the standard input and standard output streams are fully buffered if and only if the stream can be determined not to refer to an interactive device." [C99:7.19.3.7]
Explanation of observered behaviour
So, what happens is that the implementation does something platform-specific to decide whether
stdout
is going to be line-buffered. In most libc implementations, this test is done when the stream is first used.printf()
is flushed automatically.fflush()
, unless you write gobloads of data to it.printf()
, stdout acquired the line-buffered mode. When we swap out the fd to go to file, it's still line-buffered, so the data is flushed automatically.Some actual implementations
Each libc has latitude in how it interprets these requirements, since C99 doesn't specify what an "interactive device" is, nor does POSIX's stdio entry extend this (beyond requiring stderr to be open for reading).
Glibc. See filedoalloc.c:L111. Here we use
stat()
to test if the fd is a tty, and set the buffering mode accordingly. (This is called from fileops.c.)stdout
initially has a null buffer, and it's allocated on the first use of the stream based on the characteristics of fd 1.BSD libc. Very similar, but much cleaner code to follow! See this line in makebuf.c
You should not close the file descriptor, so remove
close(fd)
and closestdout_bak_fd
if you want the message to be print only in the file.