Removal of insecure "steghide" package

Hi,

I’d like to get some clarifying hints on what to do with the “steghide” package. See Vulnerability roundup 100: steghide-0.5.1: 1 advisory [7.5] · Issue #116923 · NixOS/nixpkgs · GitHub for details.

The package got an CVE advisory which basically states that it does not hide stuff as effective as it promises to do. Since the package has not seen upstream activity for 8 years, I suspect that there will be no bug fix. It does not have an nixpkgs maintainer either. I’d suggest to remove it altogether.

How to proceed? Please comment on:

  • Add knownVulnerabilites meta attribute to 20.09? unstable?
  • Remove it completely from unstable?
  • Remove it from unstable after the next branch-off so that it will remain in the next release?
  • If so: How do we make sure that we don’t forget it as there is no maintainer?

Cheers, Christian

2 Likes

My vote is to remove it from unstable right away.

4 Likes

I agree, should be removed from unstable as soon as possible.

To be fair, I only encountered people using steghide to make challenges in cybersecurity events.

I vote for the first option and/or to mark the package as broken?

2 Likes

are we hackers yet?

https://jjjollyjim.github.io/arewehackersyet/index.html

having lots of broken packages in nixpackages not only looks bad from a PR point of view, but even if it doesn’t get built by hydra, the nix evaluations get bigger.

If it’s not in nixpkgs, there probably more scope to to add it.

retiring libraries and things that depend on just one top level application might be a different case.

so it’s either play the package numbers vanity game, or have have as many high quality working packages as possible?

i’m not sure what it is best.

and what is broken anyway? it doesn’t actually compile? it’s a securtiy risk? the code is dead, unmaintained and is useless :-(… maybe ‘broken’ needs more definition?

There we go:

3 Likes

We removed it. If someone wants to pick up maintainership and stick it back in with appropriate vulnerability notices, go for it.