Skip to content

wireshark: 4.0.1 -> 4.0.2#210742

Merged
LeSuisse merged 1 commit intoNixOS:masterfrom
risicle:ris-wireshark-4.0.2
Jan 16, 2023
Merged

wireshark: 4.0.1 -> 4.0.2#210742
LeSuisse merged 1 commit intoNixOS:masterfrom
risicle:ris-wireshark-4.0.2

Conversation

@risicle
Copy link
Contributor

@risicle risicle commented Jan 14, 2023

Description of changes

https://nvd.nist.gov/vuln/detail/CVE-2022-4344
https://nvd.nist.gov/vuln/detail/CVE-2022-4345

pyshark tests as flaky and annoying as ever. wireshark is currently broken on x86_64 darwin so couldn't test there.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.05 Release Notes (or backporting 22.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@risicle risicle added 1.severity: security Issues which raise a security issue, or PRs that fix one backport release-22.11 labels Jan 14, 2023
@ofborg ofborg bot requested review from bjornfor and fpletz January 14, 2023 19:09
@ofborg ofborg bot added 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Jan 14, 2023
@risicle risicle marked this pull request as ready for review January 14, 2023 20:06
@risicle risicle mentioned this pull request Jan 14, 2023
13 tasks
@teto
Copy link
Member

teto commented Jan 15, 2023

Result of nixpkgs-review pr 210742 run on x86_64-linux 1

2 packages failed to build:
  • credslayer
  • python311Packages.pyshark
10 packages built:
  • compactor
  • dbmonster
  • haka
  • hfinger
  • ostinato
  • python310Packages.pyshark
  • termshark
  • tshark (wireshark-cli)
  • wifite2
  • wireshark (wireshark-qt)

Copy link
Member

@LeSuisse LeSuisse left a comment

Choose a reason for hiding this comment

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

nixpkgs-review happy on x86_64 (failure of credslayer is not caused by this change). The failure on Darwin x86_64 is due to #210961.

Result of nixpkgs-review pr 210742 run on x86_64-linux 1

1 package failed to build:
  • credslayer
11 packages built:
  • compactor
  • dbmonster
  • haka
  • hfinger
  • ostinato
  • python310Packages.pyshark
  • python311Packages.pyshark
  • termshark
  • tshark (wireshark-cli)
  • wifite2
  • wireshark (wireshark-qt)

@LeSuisse LeSuisse merged commit b77bbec into NixOS:master Jan 16, 2023
@github-actions
Copy link
Contributor

Successfully created backport PR #211109 for release-22.11.

@vcunat
Copy link
Member

vcunat commented Jan 17, 2023

Even after fixing that, x86_64-darwin is still broken: https://hydra.nixos.org/job/nixpkgs/trunk/wireshark.x86_64-darwin

EDIT: but it's weird that it seems OK in the 22.11 backport, according to OfBorg.

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

Labels

1.severity: security Issues which raise a security issue, or PRs that fix one 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants