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

When the file doesnt exists (--rsa-public-key-path), execution results in Segmentation fault #1471

Closed
declum opened this issue Feb 15, 2023 · 0 comments · Fixed by #1488
Closed

Comments

@declum
Copy link

declum commented Feb 15, 2023

Context

  • Version of iperf3:

iperf 3.9 (cJSON 1.7.13)
Linux lcm-worker-vm 5.15.0-60-generic #66-Ubuntu SMP Fri Jan 20 14:29:49 UTC 2023 x86_64
Optional features available: CPU affinity setting, IPv6 flow label, SCTP, TCP congestion algorithm setting, sendfile / zerocopy, socket pacing, authentication

iperf 3.12 (cJSON 1.7.15)
Darwin cjebakumarPF2K7.vmware.com 22.2.0 Darwin Kernel Version 22.2.0: Fri Nov 11 02:03:51 PST 2022; root:xnu-8792.61.2~4/RELEASE_ARM64_T6000 arm64
Optional features available: sendfile / zerocopy, authentication, support IPv4 don't fragment

  • Operating system (and distribution, if any): Mac 13.1 (installed by brew) / Ubuntu 22.04

Bug Report

When the file doesnt exists (--rsa-public-key-path), execution results in Segmentation fault

  • Expected Behavior
    iperf3 should show a message that the file doesn't exists. Currently, it emits Segmentation fault (as String) even for -J/--json flag. So the caller needs to have special handling.

  • Actual Behavior
    Segmentation fault

  • Steps to Reproduce
    iperf3 --get-server-output --json -c --port 5201 --username user --rsa-public-key-path /tmp/public-missing-file.pem --connect-timeout 3000 --time 5
    Password:
    Segmentation fault (core dumped)

  • Possible Solution
    stats the file, and show error if its not found.

davidBar-On added a commit to davidBar-On/iperf that referenced this issue Mar 22, 2023
@bmah888 bmah888 linked a pull request Apr 3, 2023 that will close this issue
coolshou pushed a commit to coolshou/iperf that referenced this issue Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant