Bug 267015 - [patch] net-mgmt/smokeping: fix bug in FPingContinuous.pm
Summary: [patch] net-mgmt/smokeping: fix bug in FPingContinuous.pm
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Rodrigo Osorio
URL:
Keywords: easy, patch-ready
Depends on:
Blocks:
 
Reported: 2022-10-13 13:02 UTC by Eugene Grosbein
Modified: 2022-11-04 15:31 UTC (History)
1 user (show)

See Also:
eugen: maintainer-feedback-


Attachments
proposed fix (737 bytes, patch)
2022-10-13 13:02 UTC, Eugene Grosbein
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eugene Grosbein freebsd_committer freebsd_triage 2022-10-13 13:02:43 UTC
Created attachment 237261 [details]
proposed fix

Once upon a time there was a bug #90 in Smokeping:
https://github.com/oetiker/SmokePing/issues/90

It occured because the software parses output of fping utility instead of checking its version and the output format changes in time. The bugfix was incomplete as it fixed the problem in FPing.pm but same code is in FPingContinuous.pm and same bug was not fixed there.

The problem's reported as bug #341 three weeks ago but neglected:
https://github.com/oetiker/SmokePing/issues/341

Let us add same fix as in #90 to our port to fix FPingContinuous.pm until its fixed upstream.
Comment 1 Eugene Grosbein freebsd_committer freebsd_triage 2022-10-13 13:07:29 UTC
Forgot to note, that this problem makes it silently ignore "sourceaddress" setting for a proble that may result in completely wrong path for probe packets and misleading graphs.
Comment 2 commit-hook freebsd_committer freebsd_triage 2022-11-04 15:31:04 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=55c01f69543c6336bb583857019f5278d4bf90ef

commit 55c01f69543c6336bb583857019f5278d4bf90ef
Author:     Eugene Grosbein <eugen@FreeBSD.org>
AuthorDate: 2022-11-04 15:27:04 +0000
Commit:     Eugene Grosbein <eugen@FreeBSD.org>
CommitDate: 2022-11-04 15:27:04 +0000

    net-mgmt/smokeping: fix bug in FPingContinuous.pm

    Add patch accepted upstream with bug #341.
    https://github.com/oetiker/SmokePing/issues/341

    PR:             267015
    Approved by:    rodrigo (maintainer timeout, 3 weeks)

 net-mgmt/smokeping/Makefile                                 |  2 +-
 .../patch-lib_Smokeping_probes_FPingContinuous.pm (new)     | 13 +++++++++++++
 2 files changed, 14 insertions(+), 1 deletion(-)