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

Kodi's urllib3 might be vulnerable to CVE-2021-33503 #176270

Closed
dotlambda opened this issue Jun 4, 2022 · 11 comments · Fixed by #176391
Closed

Kodi's urllib3 might be vulnerable to CVE-2021-33503 #176270

dotlambda opened this issue Jun 4, 2022 · 11 comments · Fixed by #176391
Labels
1.severity: security 9.needs: port to stable A PR needs a backport to the stable release.

Comments

@dotlambda
Copy link
Member

The version is called 1.26.4+matrix.1. That seems to be older than 1.26.5 which is the first version with a fix for https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-33503.
Can't we use python3Packages.urllib3?
cc @aanderse @cpages @edwtjo @minijackson @peterhoeg @sephalon

@aanderse
Copy link
Member

aanderse commented Jun 4, 2022

Would be nice if we could push issues like this upstream and have them fix it. Our tooling is nice and all we would have to do is backport the bump that our bot automatically creates.

Any idea if upstream has addressed this? I can probably look later tonight when I'm at a computer.

@dotlambda
Copy link
Member Author

That's the problem with such tooling. You only get security updates after upstream notices them, if ever. Thus at least one more indirection.

@aanderse
Copy link
Member

aanderse commented Jun 4, 2022

That's the problem with such tooling. You only get security updates after upstream notices them, if ever.

I guess I'm just spoiled by our (nixpkgs) tooling because all this comment made me think is that upstream could really benefit from using this type of tooling too 😉

@dotlambda
Copy link
Member Author

Is there a way to override specific plugins?
There wouldn't be much to maintain if you just use python3Packages.urllib3.

@aanderse
Copy link
Member

aanderse commented Jun 4, 2022

Straight python libs generally won't work as kodi packaging does a little wrapping. Certainly we could replicate this... but that would come at a bit of a maintenance cost too.

@dotlambda
Copy link
Member Author

We'll have to do either that or update upstream asap.

@aanderse
Copy link
Member

aanderse commented Jun 4, 2022

I'm out for the day and my android phone can't read .xz files so I can't check if upstream has fixed this or not.

Manually running the update script would let us know... if you have the capacity.

@dotlambda dotlambda added the 9.needs: port to stable A PR needs a backport to the stable release. label Jun 4, 2022
@dotlambda
Copy link
Member Author

dotlambda commented Jun 4, 2022

Manually running the update script would let us know... if you have the capacity.

How do I run it?
EDIT: nix-shell maintainers/scripts/update.nix --argstr package kodiPlugins.urllib3

@dotlambda
Copy link
Member Author

$ nix-shell maintainers/scripts/update.nix --argstr package kodiPlugins.urllib3

Going to be running update for following packages:
 - kodi-urllib3-1.26.4+matrix.1

Press Enter key to continue...

Running update for:
 - kodi-urllib3-1.26.4+matrix.1: UPDATING ...
 - kodi-urllib3-1.26.4+matrix.1: DONE.

Packages updated!

Doesn't change anything.

@dotlambda
Copy link
Member Author

Where are these add-ons maintained upstream?

@aanderse
Copy link
Member

aanderse commented Jun 5, 2022

Looks like here. It seems like kodi developers pull upstream code right into their tree and then make a few minor additions for packaging as seen here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.severity: security 9.needs: port to stable A PR needs a backport to the stable release.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants