NixOS / nixpkgs

Nix Packages collection & NixOS
MIT License
18.14k stars 14.17k forks source link

Vulnerability Roundup 7 #20078

Closed grahamc closed 8 years ago

grahamc commented 8 years ago

Here are all the vulnerabilities from https://lwn.net/Vulnerabilities since our last hunt.

Notes on the list

  1. The reports have been roughly grouped by the package name. This isn't perfect, but is intended to help identify if a whole group of reports is resolved already.
  2. Some issues will be duplicated, because it affects multiple packages. For example, there are sometimes problems that impact thunderbird, and firefox. LWN might report in one vulnerability "thunderbird firefox". These names have been split to make sure both packages get addressed.
  3. By each issue is a link to code search for the package name, and a Github search by filename. These are to help, but may not return results when we do in fact package the software. If a search doesn't turn up, please try altering the search criteria or looking in nixpkgs manually before asserting we don't have it.

Instructions:

  1. Triage a report: If we don't have the software or our version isn't vulnerable, tick the box or add a comment with the report number, stating it isn't vulnerable.
  2. Fix the issue: If we do have the software and it is vulnerable, either leave a comment on this issue saying so, even open a pull request with the fix. If you open a PR, make sure to tag this issue so we can coordinate.
  3. When an entire section is completed, move the section to the "Triaged and Resolved Issues" details block below.

Upon Completion ...

Without further ado...

Assorted (21 issues)

graphicsmagick (3 issues)

kernel (2 issues)

mariadb (2 issues)

mysql (2 issues)

potrace (2 issues)

grahamc commented 8 years ago

cc @FRidh, @fpletz, @NeQuissimus, @vcunat who participated last time.

Also:

PS: I'm unusually busy this morning, and won't be able to participate as much as normal. I'll have some time here and there. Feel free to conscript your friends to help finish this out ;)

rycee commented 8 years ago

The perl-Image-Info package is vulnerable. I will push an updated version shortly.

grahamc commented 8 years ago

@rycee Thank you for the patches and what-not! When you push a commit to a branch, can you add a comment on this issue with the package, and the sha that you fixed it in? that'll help me do the summary at the end.

grahamc commented 8 years ago

perl-Image-Info: https://github.com/NixOS/nixpkgs/commit/68f2bc8fb351065fda55c8a7b1ee6d74ba64a9a0 / f33c5f713e1aa7c780134154e8e5072ad2081921

rycee commented 8 years ago

@grahamc Sorry, I missed that. I'll make sure to include the commit hashes in the future.

fpletz commented 8 years ago

Django (not yet on LWN): 6ad14d42569ffbf214ee301aaa9f47370bc5555d, 58ad105cd43356e3de024fbf7df2d34f10d696df, b806e14a3ced762ec2b0ce162c75d400f312e897

https://www.djangoproject.com/weblog/2016/nov/01/security-releases/

We should remove Django 1.5 and 1.6 as they're not maintained upstream anymore. One of the users of Django 1.6 in nixpks, reviewboard, is maintaining a version with security patches, maybe switch to this: https://www.reviewboard.org/news/2016/11/01/new-django-1-6-11-5-security-releases/

I'm currently investigating and will open a PR shortly if all goes well.

NeQuissimus commented 8 years ago

I just took a quick look at gnutar and it needs to be changed to pull from git. But something is wrong with fetchgit, it complains about an issue in cpio of all things. Not really sure what to do.

NeQuissimus commented 8 years ago

nginx is up-to-date. I think the vulnerability is Debian-specific

vcunat commented 8 years ago

@NeQuissimus: tar may pose problems due to being involved in bootstrapping.

NeQuissimus commented 8 years ago

@vcunat This is how far I got: https://gist.github.com/NeQuissimus/9a01a2215fffba1ce789ca598486fa46

1) Changed the fetchurl to fetchgit 2) nix-build complains about a chain of things, which lead me back to fetchurl/boot.nix 3) Added a , ... argument to the function there "fixes" the issue 4) C build does not work; and I have no idea about C builds :)

grahamc commented 8 years ago

@vcunat can you help me out with fixing the tar issue? I'd love to understand how that process works.

grahamc commented 8 years ago

BTW thank you everyone for help with this roundup. I've had an incredibly busy week, and regret not being more involved. I'll do a summary on this one shortly.

grahamc commented 8 years ago

I'm closing out this issue for now, but it is important to note hydra hasn't passed in some time: https://hydra.nixos.org/build/43025591 can someone diagnose this issue and try and get a fix in?

Also, we really need to get this tar issue fixed. @vcunat If not you, who could show me the process here?

shlevy commented 8 years ago

@grahamc What's the issue with tar? I can help.

shlevy commented 8 years ago

@grahamc fix for the nss issue https://github.com/NixOS/nixpkgs/commit/80cbb8acf17cd128e834d4a11f8270b9a5197166

grahamc commented 8 years ago

Thank you for the nss patch, @shlevy! Regarding tar, it is mishandling .. in archives: https://lwn.net/Vulnerabilities/705216/ which @NeQuissimus was trying to patch: https://gist.github.com/NeQuissimus/9a01a2215fffba1ce789ca598486fa46

shlevy commented 8 years ago

Testing a fix

shlevy commented 8 years ago

https://github.com/NixOS/nixpkgs/commit/ac59e2f184fe0beeec2920dc9d6d9830813d9212 https://github.com/NixOS/nixpkgs/commit/674ebc241dde3b747e8c47a5ca1b5825cba95c38

shlevy commented 8 years ago

Will link to jobsets shortly...

shlevy commented 8 years ago

Well, I wanted to link to evals but the loop is taking too long ( @edolstra @rbvermaa everyting good with hydra?) http://hydra.nixos.org/jobset/nixos/staging http://hydra.nixos.org/jobset/nixos/staging-16.09 I'll check back in the morning.

vcunat commented 8 years ago

It does compilation during eval! http://hydra.nixos.org/jobset/nixos/staging#tabs-errors

vcunat commented 8 years ago

Using git would complicate the bootstrapping process and make it longer, so it's nicer to avoid that like shlevy did.

vcunat commented 8 years ago

I can post a few lines about (linux stdenv) bootstrapping here/somewhere, if you're still interested.

grahamc commented 8 years ago

Yes please!

shlevy commented 8 years ago

The build-during-eval is due to the cjdns test...