diff options
-rw-r--r-- | README | 3 | ||||
-rw-r--r-- | README.md | 4 | ||||
-rw-r--r-- | RELNOTES | 2 | ||||
-rw-r--r-- | src/firejail/main.c | 2 | ||||
-rw-r--r-- | src/firejail/x11.c | 7 | ||||
-rw-r--r-- | src/man/firejail.txt | 2 |
6 files changed, 8 insertions, 12 deletions
@@ -43,7 +43,7 @@ Aleksey Manevich (https://github.com/manevich) | |||
43 | - added --fix option to firecfg utility | 43 | - added --fix option to firecfg utility |
44 | - read_pid fix | 44 | - read_pid fix |
45 | - added --x11=block options | 45 | - added --x11=block options |
46 | - x11 xpra, xphyr, block profile commands | 46 | - x11 xpra, xphyr, none profile commands |
47 | - added --join-or-start command | 47 | - added --join-or-start command |
48 | - CVE-2016-7545 | 48 | - CVE-2016-7545 |
49 | Fred-Barclay (https://github.com/Fred-Barclay) | 49 | Fred-Barclay (https://github.com/Fred-Barclay) |
@@ -79,6 +79,7 @@ Fred-Barclay (https://github.com/Fred-Barclay) | |||
79 | - evince profile enhancement | 79 | - evince profile enhancement |
80 | vismir2 (https://github.com/vismir2) | 80 | vismir2 (https://github.com/vismir2) |
81 | - feh, ranger and zathura profiles | 81 | - feh, ranger and zathura profiles |
82 | - lots of profile fixes | ||
82 | graywolf (https://github.com/graywolf) | 83 | graywolf (https://github.com/graywolf) |
83 | - spelling fix | 84 | - spelling fix |
84 | Dara Adib (https://github.com/daradib) | 85 | Dara Adib (https://github.com/daradib) |
@@ -47,7 +47,7 @@ If you keep your Firejail profiles in a public repository, please give us a link | |||
47 | 47 | ||
48 | ## New command line options | 48 | ## New command line options |
49 | ````` | 49 | ````` |
50 | --x11=block | 50 | --x11=none |
51 | Blacklist /tmp/.X11-unix directory, ${HOME}/.Xauthority and file | 51 | Blacklist /tmp/.X11-unix directory, ${HOME}/.Xauthority and file |
52 | specified in ${XAUTHORITY} environment variable. Remove DISPLAY and | 52 | specified in ${XAUTHORITY} environment variable. Remove DISPLAY and |
53 | XAUTHORITY environment variables. Stop with error message if X11 | 53 | XAUTHORITY environment variables. Stop with error message if X11 |
@@ -84,7 +84,7 @@ If you keep your Firejail profiles in a public repository, please give us a link | |||
84 | 84 | ||
85 | ## New profile commands | 85 | ## New profile commands |
86 | 86 | ||
87 | x11 xpra, x11 xephyr, x11 block, allusers, join-or-start | 87 | x11 xpra, x11 xephyr, x11 none, allusers, join-or-start |
88 | 88 | ||
89 | ## New profiles | 89 | ## New profiles |
90 | 90 | ||
@@ -4,8 +4,6 @@ firejail (0.9.43) baseline; urgency=low | |||
4 | * modifs: removed man firejail-config | 4 | * modifs: removed man firejail-config |
5 | * modifs: --private-tmp whitelists /tmp/.X11-unix directory | 5 | * modifs: --private-tmp whitelists /tmp/.X11-unix directory |
6 | * modifs: Nvidia drivers added to --private-dev | 6 | * modifs: Nvidia drivers added to --private-dev |
7 | * modifs: dropping requirement for network namespace when | ||
8 | using a --x11 | ||
9 | * feature: support starting/joining sandbox is a single command | 7 | * feature: support starting/joining sandbox is a single command |
10 | (--join-or-start) | 8 | (--join-or-start) |
11 | * feature: X11 detection support for --audit | 9 | * feature: X11 detection support for --audit |
diff --git a/src/firejail/main.c b/src/firejail/main.c index 91ea523ab..c4b7ce4dc 100644 --- a/src/firejail/main.c +++ b/src/firejail/main.c | |||
@@ -2208,7 +2208,7 @@ int main(int argc, char **argv) { | |||
2208 | return 1; | 2208 | return 1; |
2209 | } | 2209 | } |
2210 | } | 2210 | } |
2211 | else if (strcmp(argv[i], "--x11=block") == 0) { | 2211 | else if (strcmp(argv[i], "--x11=none") == 0) { |
2212 | arg_x11_block = 1; | 2212 | arg_x11_block = 1; |
2213 | } | 2213 | } |
2214 | else if (strncmp(argv[i], "--join-or-start=", 16) == 0) { | 2214 | else if (strncmp(argv[i], "--join-or-start=", 16) == 0) { |
diff --git a/src/firejail/x11.c b/src/firejail/x11.c index 36023e067..def1dd324 100644 --- a/src/firejail/x11.c +++ b/src/firejail/x11.c | |||
@@ -52,7 +52,6 @@ static int x11_check_xephyr(void) { | |||
52 | return 1; | 52 | return 1; |
53 | } | 53 | } |
54 | 54 | ||
55 | #if 0 | ||
56 | // check for X11 abstract sockets | 55 | // check for X11 abstract sockets |
57 | static int x11_abstract_sockets_present(void) { | 56 | static int x11_abstract_sockets_present(void) { |
58 | char *path; | 57 | char *path; |
@@ -77,7 +76,6 @@ static int x11_abstract_sockets_present(void) { | |||
77 | 76 | ||
78 | return 0; | 77 | return 0; |
79 | } | 78 | } |
80 | #endif | ||
81 | 79 | ||
82 | static int random_display_number(void) { | 80 | static int random_display_number(void) { |
83 | int i; | 81 | int i; |
@@ -598,18 +596,17 @@ void x11_start(int argc, char **argv) { | |||
598 | void x11_block(void) { | 596 | void x11_block(void) { |
599 | #ifdef HAVE_X11 | 597 | #ifdef HAVE_X11 |
600 | mask_x11_abstract_socket = 1; | 598 | mask_x11_abstract_socket = 1; |
601 | #if 0 | 599 | |
602 | // check abstract socket presence and network namespace options | 600 | // check abstract socket presence and network namespace options |
603 | if ((!arg_nonetwork && !cfg.bridge0.configured && !cfg.interface0.configured) | 601 | if ((!arg_nonetwork && !cfg.bridge0.configured && !cfg.interface0.configured) |
604 | && x11_abstract_sockets_present()) { | 602 | && x11_abstract_sockets_present()) { |
605 | fprintf(stderr, "ERROR: --x11=block specified, but abstract X11 socket still accessible.\n" | 603 | fprintf(stderr, "ERROR: --x11=none specified, but abstract X11 socket still accessible.\n" |
606 | "Additional setup required. To block abstract X11 socket you can either:\n" | 604 | "Additional setup required. To block abstract X11 socket you can either:\n" |
607 | " * use network namespace in firejail (--net=none, --net=...)\n" | 605 | " * use network namespace in firejail (--net=none, --net=...)\n" |
608 | " * add \"-nolisten local\" to xserver options\n" | 606 | " * add \"-nolisten local\" to xserver options\n" |
609 | " (eg. to your display manager config, or /etc/X11/xinit/xserverrc)\n"); | 607 | " (eg. to your display manager config, or /etc/X11/xinit/xserverrc)\n"); |
610 | exit(1); | 608 | exit(1); |
611 | } | 609 | } |
612 | #endif | ||
613 | 610 | ||
614 | // blacklist sockets | 611 | // blacklist sockets |
615 | profile_check_line("blacklist /tmp/.X11-unix", 0, NULL); | 612 | profile_check_line("blacklist /tmp/.X11-unix", 0, NULL); |
diff --git a/src/man/firejail.txt b/src/man/firejail.txt index 3ceb21e78..aadc54677 100644 --- a/src/man/firejail.txt +++ b/src/man/firejail.txt | |||
@@ -1713,7 +1713,7 @@ Example: | |||
1713 | $ firejail \-\-x11=xephyr --net=eth0 openbox | 1713 | $ firejail \-\-x11=xephyr --net=eth0 openbox |
1714 | 1714 | ||
1715 | .TP | 1715 | .TP |
1716 | \fB\-\-x11=block | 1716 | \fB\-\-x11=none |
1717 | Blacklist /tmp/.X11-unix directory, ${HOME}/.Xauthority and file specified in ${XAUTHORITY} environment variable. | 1717 | Blacklist /tmp/.X11-unix directory, ${HOME}/.Xauthority and file specified in ${XAUTHORITY} environment variable. |
1718 | Remove DISPLAY and XAUTHORITY environment variables. | 1718 | Remove DISPLAY and XAUTHORITY environment variables. |
1719 | Stop with error message if X11 abstract socket will be accessible in jail. | 1719 | Stop with error message if X11 abstract socket will be accessible in jail. |