Skip to content

Commit

Permalink
wireshark: 4.2.6 -> 4.4.2 (#344914)
Browse files Browse the repository at this point in the history
  • Loading branch information
fpletz authored Dec 3, 2024
2 parents def6c2f + 90c4ec9 commit 6d8ddd8
Show file tree
Hide file tree
Showing 4 changed files with 30 additions and 18 deletions.
12 changes: 6 additions & 6 deletions pkgs/applications/networking/sniffers/wireshark/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
, libpcap
, libsmi
, libssh
, lua5
, lua5_4
, lz4
, makeWrapper
, minizip
Expand All @@ -44,7 +44,7 @@
, speexdsp
, SystemConfiguration
, wrapGAppsHook3
, zlib
, zlib-ng
, zstd

, withQt ? true
Expand All @@ -57,15 +57,15 @@ assert withQt -> qt6 != null;

stdenv.mkDerivation rec {
pname = "wireshark-${if withQt then "qt" else "cli"}";
version = "4.2.8";
version = "4.4.2";

outputs = [ "out" "dev" ];

src = fetchFromGitLab {
repo = "wireshark";
owner = "wireshark";
rev = "v${version}";
hash = "sha256-QnBETFkYoeBTQFV8g2c/dZjgCXaMtFi1MQUgmkOool8=";
hash = "sha256-qeMaj8kRGG1NlDb5j4M/Za2M2Ohh2qhXbzBtQGjrCSo=";
};

patches = [
Expand Down Expand Up @@ -107,7 +107,7 @@ stdenv.mkDerivation rec {
libpcap
libsmi
libssh
lua5
lua5_4
lz4
minizip
nghttp2
Expand All @@ -118,7 +118,7 @@ stdenv.mkDerivation rec {
snappy
spandsp3
speexdsp
zlib
zlib-ng
zstd
] ++ lib.optionals withQt (with qt6; [
qt5compat
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,19 +19,21 @@ EDITED by esclear for wireshark 4.0

EDITED by paveloom for wireshark 4.2

EDITED by pabloaul for wireshark 4.4

Signed-off-by: Franz Pletz <[email protected]>
---
capture/capture_sync.c | 13 ++++++++++---
1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/capture/capture_sync.c b/capture/capture_sync.c
index 01e9510a27..e439098298 100644
index 946dc810db..ef0b6e12cd 100644
--- a/capture/capture_sync.c
+++ b/capture/capture_sync.c
@@ -225,8 +225,15 @@ init_pipe_args(int *argc) {
@@ -243,8 +243,15 @@ init_pipe_args(int *argc) {
char *exename;
char **argv;

- /* Find the absolute path of the dumpcap executable. */
- exename = get_executable_path("dumpcap");
+ /* NixOS patch: Look for dumpcap in PATH first, because there may be a
Expand All @@ -46,14 +48,12 @@ index 01e9510a27..e439098298 100644
if (exename == NULL) {
return NULL;
}
@@ -533,7 +540,7 @@ sync_pipe_open_command(char* const argv[], int *data_read_fd,
dup2(sync_pipe[PIPE_WRITE], 2);
ws_close(sync_pipe[PIPE_READ]);
ws_close(sync_pipe[PIPE_WRITE]);
@@ -596,7 +603,7 @@ sync_pipe_open_command(char **argv, int *data_read_fd,
snprintf(sync_id, ARGV_NUMBER_LEN, "%d", sync_pipe[PIPE_WRITE]);
argv = sync_pipe_add_arg(argv, &argc, sync_id);
#endif
- execv(argv[0], argv);
+ execvp(argv[0], argv);
sync_pipe_write_int_msg(2, SP_EXEC_FAILED, errno);

/* Exit with "_exit()", so that we don't close the connection
--
2.42.0
sync_pipe_write_int_msg(sync_pipe[PIPE_WRITE], SP_EXEC_FAILED, errno);

/* Exit with "_exit()", so that we don't close the connection
3 changes: 3 additions & 0 deletions pkgs/by-name/cr/credslayer/package.nix
Original file line number Diff line number Diff line change
Expand Up @@ -55,5 +55,8 @@ python3.pkgs.buildPythonApplication rec {
homepage = "https://github.com/ShellCode33/CredSLayer";
license = with licenses; [ gpl3Only ];
maintainers = with maintainers; [ fab ];
# broken due to wireshark 4.4 bump
# see: https://github.com/NixOS/nixpkgs/pull/344914
broken = true;
};
}
9 changes: 9 additions & 0 deletions pkgs/development/python-modules/pyshark/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
appdirs,
buildPythonPackage,
fetchFromGitHub,
fetchpatch,
lxml,
packaging,
py,
Expand All @@ -30,6 +31,14 @@ buildPythonPackage rec {
# `stripLen` does not seem to work here
patchFlags = [ "-p2" ];

patches = [
# fixes capture test
(fetchpatch {
url = "https://github.com/KimiNewt/pyshark/commit/7142c5bf88abcd4c65c81052a00226d6155dda42.patch";
hash = "sha256-Ti7cwRyYSbF4a4pEEV9FntNevkV/JVXNqACQWzoma7g=";
})
];

sourceRoot = "${src.name}/src";

# propagate wireshark, so pyshark can find it when used
Expand Down

0 comments on commit 6d8ddd8

Please sign in to comment.