From a239728ec064666025de2723997d87b176d57fd6 Mon Sep 17 00:00:00 2001 From: Maciej Zenczykowski Date: Mon, 4 Apr 2011 15:30:32 +0200 Subject: mark newly opened fds as FD_CLOEXEC (close on exec) (This is iptables-1.4.3.1-cloexec.patch from RedHat iptables.src.rpm) Signed-off-by: Maciej Zenczykowski Signed-off-by: Patrick McHardy --- extensions/libipt_realm.c | 2 +- ip6tables-restore.c | 2 +- ip6tables-save.c | 2 +- iptables-restore.c | 2 +- iptables-save.c | 2 +- iptables-xml.c | 2 +- xtables.c | 11 +++++++++++ 7 files changed, 17 insertions(+), 6 deletions(-) diff --git a/extensions/libipt_realm.c b/extensions/libipt_realm.c index a2505706..17b1754e 100644 --- a/extensions/libipt_realm.c +++ b/extensions/libipt_realm.c @@ -49,7 +49,7 @@ static void load_realms(void) int id; struct realmname *oldnm = NULL, *newnm = NULL; - fil = fopen(rfnm, "r"); + fil = fopen(rfnm, "re"); if (!fil) { rdberr = 1; return; diff --git a/ip6tables-restore.c b/ip6tables-restore.c index e9a130fa..10c3acfc 100644 --- a/ip6tables-restore.c +++ b/ip6tables-restore.c @@ -168,7 +168,7 @@ int main(int argc, char *argv[]) } if (optind == argc - 1) { - in = fopen(argv[optind], "r"); + in = fopen(argv[optind], "re"); if (!in) { fprintf(stderr, "Can't open %s: %s\n", argv[optind], strerror(errno)); diff --git a/ip6tables-save.c b/ip6tables-save.c index dc189e99..c3b8ec0d 100644 --- a/ip6tables-save.c +++ b/ip6tables-save.c @@ -41,7 +41,7 @@ static int for_each_table(int (*func)(const char *tablename)) FILE *procfile = NULL; char tablename[IP6T_TABLE_MAXNAMELEN+1]; - procfile = fopen("/proc/net/ip6_tables_names", "r"); + procfile = fopen("/proc/net/ip6_tables_names", "re"); if (!procfile) return ret; diff --git a/iptables-restore.c b/iptables-restore.c index 31ce52bf..c2cc58c8 100644 --- a/iptables-restore.c +++ b/iptables-restore.c @@ -174,7 +174,7 @@ main(int argc, char *argv[]) } if (optind == argc - 1) { - in = fopen(argv[optind], "r"); + in = fopen(argv[optind], "re"); if (!in) { fprintf(stderr, "Can't open %s: %s\n", argv[optind], strerror(errno)); diff --git a/iptables-save.c b/iptables-save.c index 3bcf4227..3e3ec43c 100644 --- a/iptables-save.c +++ b/iptables-save.c @@ -39,7 +39,7 @@ static int for_each_table(int (*func)(const char *tablename)) FILE *procfile = NULL; char tablename[IPT_TABLE_MAXNAMELEN+1]; - procfile = fopen("/proc/net/ip_tables_names", "r"); + procfile = fopen("/proc/net/ip_tables_names", "re"); if (!procfile) return ret; diff --git a/iptables-xml.c b/iptables-xml.c index 8d670565..57c7486d 100644 --- a/iptables-xml.c +++ b/iptables-xml.c @@ -651,7 +651,7 @@ main(int argc, char *argv[]) } if (optind == argc - 1) { - in = fopen(argv[optind], "r"); + in = fopen(argv[optind], "re"); if (!in) { fprintf(stderr, "Can't open %s: %s", argv[optind], strerror(errno)); diff --git a/xtables.c b/xtables.c index 2f00e394..352963f4 100644 --- a/xtables.c +++ b/xtables.c @@ -300,6 +300,11 @@ static char *get_modprobe(void) procfile = open(PROC_SYS_MODPROBE, O_RDONLY); if (procfile < 0) return NULL; + if (fcntl(procfile, F_SETFD, FD_CLOEXEC) == -1) { + fprintf(stderr, "Could not set close on exec: %s\n", + strerror(errno)); + exit(1); + } ret = malloc(PROCFILE_BUFSIZ); if (ret) { @@ -697,6 +702,12 @@ static int compatible_revision(const char *name, uint8_t revision, int opt) exit(1); } + if (fcntl(sockfd, F_SETFD, FD_CLOEXEC) == -1) { + fprintf(stderr, "Could not set close on exec: %s\n", + strerror(errno)); + exit(1); + } + xtables_load_ko(xtables_modprobe_program, true); strcpy(rev.name, name); -- cgit v1.2.3