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

Vulnerability Roundup 14 #21289

Closed
31 of 33 tasks
grahamc opened this issue Dec 19, 2016 · 41 comments
Closed
31 of 33 tasks

Vulnerability Roundup 14 #21289

grahamc opened this issue Dec 19, 2016 · 41 comments

Comments

@grahamc
Copy link
Member

grahamc commented Dec 19, 2016

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

This roundup is different from previous' due to Christmas: http://lists.science.uu.nl/pipermail/nix-dev/2016-December/022367.html

I will update this issue tomorrow and Wednesday with new issues.

cc: @NeQuissimus @bachp @domenkozar @makefu.

Note: The list of people CC'd on this issue participated in the last
roundup. If you participate on this roundup, I'll cc you on the next
one. If you don't participate in the next one, you won't be CC'd on
the one after that. If you would like to be CC'd on the next roundup,
add a comment to the most recent vulnerability roundup.
If you would like to be CC'd on all roundups, leave a comment and
tell @grahamc so.

Permanent CC's: @joepie91, @phanimahesh, @NixOS/security-notifications
(if you no longer want to be CC'd, ask to be removed from this list)

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.
  4. This issue is created by https://github.com/NixOS/security

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 (18 issues)

firefox (3 issues)

game-music-emu (2 issues)

kernel (2 issues)

tomcat (2 issues)

xen (2 issues)

@joachifm
Copy link
Contributor

A fix for #709669 was included in 4.8.14

@phanimahesh
Copy link
Contributor

Commit 86cf682 fixes #709468 firefox: denial of service. Can be marked as done.

@grahamc
Copy link
Member Author

grahamc commented Dec 20, 2016

libupnp is actually pupnp and can be found here: http://pupnp.sourceforge.net/ where it says 1.6.21 is out, but the downloads are all for 1.6.20. If anyone wants to investigate that, please!

@grahamc
Copy link
Member Author

grahamc commented Dec 20, 2016

most needs updating to 5.0.0a and application of patches from debian, I think: https://security-tracker.debian.org/tracker/source-package/most

@grahamc
Copy link
Member Author

grahamc commented Dec 20, 2016

Looks like Xen needs more patches. Also, our Xen is running out of time. @michalpalka -- you seem to open issues about Xen, would you like to try upgrading Xen?

@michalpalka
Copy link

@grahamc My schedule is full for the next 2 days, but will look at it on Friday

@grahamc
Copy link
Member Author

grahamc commented Dec 20, 2016

That will be really helpful. Thank you so much, @michalpalka!

@grahamc
Copy link
Member Author

grahamc commented Dec 20, 2016

We should quite likely drop samba3.

@grahamc
Copy link
Member Author

grahamc commented Dec 20, 2016

../auth/kerberos/kerberos_pac.c: In function 'check_pac_checksum':
../auth/kerberos/kerberos_pac.c:46:7: error: 'CKSUMTYPE_HMAC_SHA1_96_AES_256' undeclared (first use in this function)
  case CKSUMTYPE_HMAC_SHA1_96_AES_256:
       ^
../auth/kerberos/kerberos_pac.c:46:7: note: each undeclared identifier is reported only once for each function it appears in
../auth/kerberos/kerberos_pac.c:52:7: error: 'CKSUMTYPE_HMAC_SHA1_96_AES_128' undeclared (first use in this function)
  case CKSUMTYPE_HMAC_SHA1_96_AES_128:
       ^
Waf: Leaving directory `/tmp/nix-build-samba-4.4.8.drv-0/samba-4.4.8/bin'
Build failed:  -> task failed (err #1): 
	{task: cc kerberos_pac.c -> kerberos_pac_1.o}
make: *** [Makefile:8: all] Error 1

@abbradar, @wkennington any ideas on what is wrong with samba here?

@abbradar
Copy link
Member

@grahamc I've tried to update libkrb5 and Cyrus-SASL -- no luck so far. I'll spend more time on this later since this is a security issue but have no idea what happens. I left Samba 4.5.3 to build in the background but this is not an option for the release, isn't it?...

@grahamc
Copy link
Member Author

grahamc commented Dec 20, 2016

Thank you for looking, @abbradar. Unfortunately backporting an update to 4.5.x wouldn't be good. I definitely wouldn't mind seeing unstable updated though, especially before 17.03 goes stable ( @michalpalka -- that is when I'd like Xen to be upgraded at the latest, hopefully)

@abbradar
Copy link
Member

Samba, libkrb5 and Cyrus-SASL updates are in staging, because Cyrus-SASL is a systemd dependency. We now need to determine how to build new 4.4.* -- I'm on it but with no ideas currently.

@bachp
Copy link
Member

bachp commented Dec 20, 2016

I did some investigation on libupnp. There is a 1.6.21 tag available but no tarball.

@grahamc
Copy link
Member Author

grahamc commented Dec 20, 2016

@abbradar can you link each of those commits here to make it easier to keep track / ensure they get backported?

@abbradar
Copy link
Member

b0a1028 covers Samba in staging. Others don't have a security issue assigned IIUC. I want to avoid having it backported as is now -- instead I'll try to have a maintenance release building.

@abbradar
Copy link
Member

16.09 is covered by b2e80a5

@LnL7
Copy link
Member

LnL7 commented Dec 20, 2016

This might work for libupnp #21317.

@grahamc
Copy link
Member Author

grahamc commented Dec 21, 2016

Just updated the list with more vulnerabilities.

@grahamc
Copy link
Member Author

grahamc commented Dec 21, 2016

I suspect we need to apply patches to xen, see: https://xenbits.xen.org/xsa/advisory-200.html and: an enterprising contributor may go back through old advisories and see if we missed anything :)

@grahamc
Copy link
Member Author

grahamc commented Dec 21, 2016

We don't run debian's version of most, so we don't need patches.

@grahamc grahamc mentioned this issue Dec 21, 2016
7 tasks
@grahamc
Copy link
Member Author

grahamc commented Dec 21, 2016

I just push my roundup branch which contains fixes for everything but the html5lib, will merge shortly.

grahamc pushed a commit to grahamc/nixpkgs that referenced this issue Dec 21, 2016
Switching to git tags means we don't get pre-generated configure
scripts. Thusly, run bootstrap ourselves.

For NixOS#21289
For CVE-2016-8863 (remote code execution)
grahamc pushed a commit that referenced this issue Dec 21, 2016
Switching to git tags means we don't get pre-generated configure
scripts. Thusly, run bootstrap ourselves.

For #21289
For CVE-2016-8863 (remote code execution)

(cherry picked from commit 0d3f0f0)
@grahamc
Copy link
Member Author

grahamc commented Dec 21, 2016

Merged and backported the branch. Just updated this issue with new vulnerabilities. :)

(this issue is like the gift that keeps giving!)

@grahamc
Copy link
Member Author

grahamc commented Dec 21, 2016

@the-kenny can you update flightgear?

@grahamc
Copy link
Member Author

grahamc commented Dec 21, 2016

Just pushed patches for zlib to staging, will push to 16.09 shortly.

@grahamc
Copy link
Member Author

grahamc commented Dec 21, 2016

@wkennington can you patch ceph? We're pretty old, I might mark it as broken otherwise.

@bachp
Copy link
Member

bachp commented Dec 21, 2016

@grahamc The libupnp tarball would be available now: https://sourceforge.net/projects/pupnp/files/

@grahamc
Copy link
Member Author

grahamc commented Dec 21, 2016

Thank you, @bachp I already switched to their github mirror since it already had a good tag.

@the-kenny
Copy link
Contributor

the-kenny commented Dec 22, 2016 via email

@phanimahesh
Copy link
Contributor

Sorry, I couldn't get around to finishing the html5lib update, got busy with work-related issues.
It requires a few dependencies (webencodings, and others I don't recollect) that have to be added to nixpkgs. My schedule for today and tomorrow is chaotic, not sure if I'll be able to clean it up. If anyone can, please comment on the issue and pitch in. I'll get to it as soon as I can make some time for it.

@the-kenny
Copy link
Contributor

the-kenny commented Dec 24, 2016 via email

@Mic92
Copy link
Member

Mic92 commented Dec 24, 2016

@the-kenny does neither not look like a critical network-facing service to me nor a serious security issue (I would not expect that malicious Nasal scripts are sent as email attachments any time soon). On the other hand it does not look the typical application, where people would expect the version to be stable all the time. I think both approaches (upgrading or not upgrading) are fine.

FRidh pushed a commit that referenced this issue Dec 25, 2016
fixes LWN vuln:709146
ref #21289: Vulnerability roundup 14
@grahamc
Copy link
Member Author

grahamc commented Dec 25, 2016

The PR from f3287b0 needs to be backported.

@grahamc
Copy link
Member Author

grahamc commented Dec 25, 2016

Also, the html5lib changes would be good to backport.

@FRidh -- thank you for merging that!

@bjornfor -- I agree, I try really hard to test all my changes. Even still, a few mistakes come through. I always feel bad about it.

@FRidh
Copy link
Member

FRidh commented Dec 25, 2016

@grahamc the new html5lib has quite some extra dependencies. I'll have a look at it.

@bjornfor
Copy link
Contributor

@bjornfor -- I agree, I try really hard to test all my changes. Even still, a few mistakes come through. I always feel bad about it.

It's frustrating when breakage happens, but I do appreciate the work you and everyone else put into keeping NixOS updated and secure :-)

FRidh pushed a commit that referenced this issue Dec 25, 2016
fixes LWN vuln:709146
ref #21289: Vulnerability roundup 14

(cherry picked from commit a737eff)
@FRidh
Copy link
Member

FRidh commented Dec 25, 2016

We now have the latest html5lib in stable as well.

@vcunat
Copy link
Member

vcunat commented Dec 25, 2016

It seems like openssh has fallen through the cracks. I didn't manage to fully fix it (yet) 661b5a9.

@grahamc
Copy link
Member Author

grahamc commented Dec 25, 2016

@vcunat openssh hasn't fallen through the cracks, it hadn't been released yet when this was created. I just checked, it'll appear on next week's.

@vcunat
Copy link
Member

vcunat commented Dec 25, 2016

Oh, now I see the tarball timestamp is Dec.19; I originally misread it as ~week earlier.

@grahamc
Copy link
Member Author

grahamc commented Dec 26, 2016

I'm going to close out what we have and let the remainders come back next run.

@grahamc
Copy link
Member Author

grahamc commented Dec 26, 2016

Good work, everyone - thank you all!

@grahamc grahamc closed this as completed Dec 26, 2016
@grahamc grahamc mentioned this issue Dec 28, 2016
23 tasks
adrianpk added a commit to adrianpk/nixpkgs that referenced this issue May 31, 2024
Switching to git tags means we don't get pre-generated configure
scripts. Thusly, run bootstrap ourselves.

For NixOS#21289
For CVE-2016-8863 (remote code execution)

(cherry picked from commit 0d3f0f0)
adrianpk added a commit to adrianpk/nixpkgs that referenced this issue May 31, 2024
fixes LWN vuln:709146
ref NixOS#21289: Vulnerability roundup 14

(cherry picked from commit a737eff)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests