CVSROOT: /cvs
Module name: ports
Changes by: b...@cvs.openbsd.org 2023/04/26 15:57:38
Modified files:
net/mcast-proxy: Makefile distinfo
Removed files:
net/mcast-proxy/patches:
patch-usr_sbin_mcast-proxy_mcast-proxy_c
Log message:
Switch upstream of mcast-proxy
Upstream (Esdenera, from Reyk Floeter) has not touched this project in
~6 years, and is not responsive, i.e., does not react on PRs. Instead of
patching mcast-proxy point GH_* to a different account, which is based
on the latest release of mcast-proxy and incorporates the following:
Fixes:
- Allow upstream to receive query and report IGMP messages. RFC4605
section 4.1 states "The proxy device performs the host portion of the
IGMP/MLD protocol on the upstream interface", and RFC2236 section 6
states "All other events, such as receiving invalid IGMP messages, or
IGMP messages other than Query or Report, are ignored in all states"
(this results in patch being dropped);
- Fix calculation of IGMP_GROUP_MEMBERSHIP_INTERVAL;
- Use correct interval values. RFC 2236 states "The Max Response Time
inserted into the periodic General Queries. Default: 100 (10
seconds)";
- Do not add upstream to a group list because it will never be removed
from this list, and remain on the routing table;
- Use SLIST_FOREACH_SAFE as entry is free'd;
- Fix scan-build warning [deadcode.DeadStores].
Features:
- Use unveil(2) "/" with no permissions instead of chroot(2)/chdir(2).
Nits:
- Fix spacing/indentation;
- Forgotten rename of igmp-proxy to mcast-proxy;
- Use *_FOREACH_SAFE macros instead of hand rolled loops.
If/when Reyk resurfaces, I can work with him to merge things back.
Tested by Joel Knight and Paul de Weerd (MAINTAINER). OK maintainer
