Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

transmission: fix RCE via dns rebinding attach #33874

Merged

Conversation

andir
Copy link
Member

@andir andir commented Jan 14, 2018

Motivation for this change

For further details see [1] & [2].

[1] transmission/transmission#468
[2] http://www.openwall.com/lists/oss-security/2018/01/12/1

(cherry picked from commit 50f48fc)

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@andir
Copy link
Member Author

andir commented Jan 14, 2018

@GrahamcOfBorg test bittorrent

Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Failure for system: x86_64-linux

killing client2 (pid 144)
killing tracker (pid 155)
killing client1 (pid 165)
killing router (pid 177)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/tmp/nix-build-vm-test-run-bittorrent.drv-0/vde1.ctl': Directory not empty
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/tmp/nix-build-vm-test-run-bittorrent.drv-0/vde2.ctl': Directory not empty
builder for ‘/nix/store/f3bv7k4advq51x4zpi168gx53za13mw2-vm-test-run-bittorrent.drv’ failed with exit code 4
error: build of ‘/nix/store/f3bv7k4advq51x4zpi168gx53za13mw2-vm-test-run-bittorrent.drv’ failed

@andir
Copy link
Member Author

andir commented Jan 14, 2018

@GrahamcOfBorg build transmission

Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Success for system: x86_64-darwin

make[2]: Entering directory '/private/tmp/nix-build-transmission-2.92.drv-0/transmission-2.92'
make[2]: Nothing to be done for 'install-exec-am'.
make[2]: Nothing to be done for 'install-data-am'.
make[2]: Leaving directory '/private/tmp/nix-build-transmission-2.92.drv-0/transmission-2.92'
make[1]: Leaving directory '/private/tmp/nix-build-transmission-2.92.drv-0/transmission-2.92'
post-installation fixup
gzipping man pages under /nix/store/v5apahijqm42raxiinhgbh8133f8vxnf-transmission-2.92/share/man/
stripping (with flags -S) in /nix/store/v5apahijqm42raxiinhgbh8133f8vxnf-transmission-2.92/bin
patching script interpreter paths in /nix/store/v5apahijqm42raxiinhgbh8133f8vxnf-transmission-2.92
/nix/store/v5apahijqm42raxiinhgbh8133f8vxnf-transmission-2.92

Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Success for system: x86_64-linux

shrinking /nix/store/wwr3c44vxhsyrkddjdrx5r0cqhia3fgk-transmission-2.92/bin/transmission-show
shrinking /nix/store/wwr3c44vxhsyrkddjdrx5r0cqhia3fgk-transmission-2.92/bin/transmission-daemon
shrinking /nix/store/wwr3c44vxhsyrkddjdrx5r0cqhia3fgk-transmission-2.92/bin/transmission-edit
shrinking /nix/store/wwr3c44vxhsyrkddjdrx5r0cqhia3fgk-transmission-2.92/bin/transmission-create
shrinking /nix/store/wwr3c44vxhsyrkddjdrx5r0cqhia3fgk-transmission-2.92/bin/transmission-cli
gzipping man pages under /nix/store/wwr3c44vxhsyrkddjdrx5r0cqhia3fgk-transmission-2.92/share/man/
stripping (with flags -S) in /nix/store/wwr3c44vxhsyrkddjdrx5r0cqhia3fgk-transmission-2.92/bin 
patching script interpreter paths in /nix/store/wwr3c44vxhsyrkddjdrx5r0cqhia3fgk-transmission-2.92
checking for references to /tmp/nix-build-transmission-2.92.drv-0 in /nix/store/wwr3c44vxhsyrkddjdrx5r0cqhia3fgk-transmission-2.92...
/nix/store/wwr3c44vxhsyrkddjdrx5r0cqhia3fgk-transmission-2.92

@andir andir merged commit 746b3a5 into NixOS:release-17.09 Jan 14, 2018
@andir andir deleted the 17.09/transmission-dns-rebinding-rce branch January 14, 2018 23:54
Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Success for system: aarch64-linux

shrinking /nix/store/qsd39067d0mx4qsb0vczbvj2ax19kqzq-transmission-2.92/bin/transmission-remote
shrinking /nix/store/qsd39067d0mx4qsb0vczbvj2ax19kqzq-transmission-2.92/bin/transmission-daemon
shrinking /nix/store/qsd39067d0mx4qsb0vczbvj2ax19kqzq-transmission-2.92/bin/transmission-show
shrinking /nix/store/qsd39067d0mx4qsb0vczbvj2ax19kqzq-transmission-2.92/bin/transmission-edit
shrinking /nix/store/qsd39067d0mx4qsb0vczbvj2ax19kqzq-transmission-2.92/bin/transmission-create
gzipping man pages under /nix/store/qsd39067d0mx4qsb0vczbvj2ax19kqzq-transmission-2.92/share/man/
stripping (with flags -S) in /nix/store/qsd39067d0mx4qsb0vczbvj2ax19kqzq-transmission-2.92/bin
patching script interpreter paths in /nix/store/qsd39067d0mx4qsb0vczbvj2ax19kqzq-transmission-2.92
checking for references to /build in /nix/store/qsd39067d0mx4qsb0vczbvj2ax19kqzq-transmission-2.92...
/nix/store/qsd39067d0mx4qsb0vczbvj2ax19kqzq-transmission-2.92

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants