bpfilter: don't pass O_CREAT when opening console for debug
authorJakub Kicinski <jakub.kicinski@netronome.com>
Thu, 24 May 2018 07:41:19 +0000 (00:41 -0700)
committerDavid S. Miller <davem@davemloft.net>
Thu, 24 May 2018 13:36:49 +0000 (09:36 -0400)
Passing O_CREAT (00000100) to open means we should also pass file
mode as the third parameter.  Creating /dev/console as a regular
file may not be helpful anyway, so simply drop the flag when
opening debug_fd.

Fixes: d2ba09c17a06 ("net: add skeleton of bpfilter kernel module")
Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bpfilter/main.c

index 81bbc16..1317f10 100644 (file)
@@ -55,7 +55,7 @@ static void loop(void)
 
 int main(void)
 {
-       debug_fd = open("/dev/console", 00000002 | 00000100);
+       debug_fd = open("/dev/console", 00000002);
        dprintf(debug_fd, "Started bpfilter\n");
        loop();
        close(debug_fd);