diff options
author | Hans-Werner Hilse <hwhilse@gmail.com> | 2015-06-11 11:29:19 +0200 |
---|---|---|
committer | Richard Weinberger <richard@nod.at> | 2015-06-25 22:42:19 +0200 |
commit | f9bb3b5947c507d402eecbffabb8fb0864263ad1 (patch) | |
tree | 82e1167c87b395d3bcb2ef3cd1e73236d1cf9527 /arch/um/drivers/harddog_user.c | |
parent | um: Do not use __ptr_t type for stack_t's .ss pointer (diff) | |
download | linux-f9bb3b5947c507d402eecbffabb8fb0864263ad1.tar.xz linux-f9bb3b5947c507d402eecbffabb8fb0864263ad1.zip |
um: Do not use stdin and stdout identifiers for struct members
stdin, stdout and stderr are macros according to C89/C99.
Thus do not use them as struct member identifiers to avoid
bad results from macro expansion.
Signed-off-by: Hans-Werner Hilse <hwhilse@gmail.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
Diffstat (limited to 'arch/um/drivers/harddog_user.c')
-rw-r--r-- | arch/um/drivers/harddog_user.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/arch/um/drivers/harddog_user.c b/arch/um/drivers/harddog_user.c index f99b32a4dbff..3aa8b0d52a48 100644 --- a/arch/um/drivers/harddog_user.c +++ b/arch/um/drivers/harddog_user.c @@ -9,8 +9,8 @@ #include <os.h> struct dog_data { - int stdin; - int stdout; + int stdin_fd; + int stdout_fd; int close_me[2]; }; @@ -18,11 +18,11 @@ static void pre_exec(void *d) { struct dog_data *data = d; - dup2(data->stdin, 0); - dup2(data->stdout, 1); - dup2(data->stdout, 2); - close(data->stdin); - close(data->stdout); + dup2(data->stdin_fd, 0); + dup2(data->stdout_fd, 1); + dup2(data->stdout_fd, 2); + close(data->stdin_fd); + close(data->stdout_fd); close(data->close_me[0]); close(data->close_me[1]); } @@ -49,8 +49,8 @@ int start_watchdog(int *in_fd_ret, int *out_fd_ret, char *sock) goto out_close_in; } - data.stdin = out_fds[0]; - data.stdout = in_fds[1]; + data.stdin_fd = out_fds[0]; + data.stdout_fd = in_fds[1]; data.close_me[0] = out_fds[1]; data.close_me[1] = in_fds[0]; |