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

openwrt-22.03: batctl: Merge bugfixes from 2024.0 #1042

Merged
merged 2 commits into from
Feb 4, 2024

Conversation

ecsv
Copy link
Contributor

@ecsv ecsv commented Feb 2, 2024

Maintainer: @simonwunderlich
Compile tested: x86_64
Run tested: x86_64

Description:

  • tcpdump: Fix missing sanity check for batman-adv header
  • tcpdump: Add missing throughput header length check
  • tcpdump: Fix IPv4 header length check
  • tcpdump: Add missing ICMPv6 Neighbor Advert length check
  • tcpdump: Add missing ICMPv6 Neighbor Solicit length check
  • tcpdump: Fix ICMPv4 inner IPv4 header length check

Copy link

github-actions bot commented Feb 3, 2024

Please do no longer set PKG_RELEASE to AUTORELEASE as the
feature is deprecated. Please use an integer instead. Below is a
list of affected packages including correct PKG_RELEASE:

  • batctl/Makefile: PKG_RELEASE:=5

Automatically compute and substitute current values for all
$(AUTORELEASE) instances as this feature is deprecated and shouldn't be
used. Since commit 2584b0cecae ("CI: deprecate $(AUTORELEASE) via
comments"), this is even announced by the CI

The following temporary change was made to the core:

    diff --git a/rules.mk b/rules.mk
    index 57d7995d4fa8..f16367de87a8 100644
    --- a/rules.mk
    +++ b/rules.mk
    @@ -429,7 +429,7 @@ endef
     abi_version_str = $(subst -,,$(subst _,,$(subst .,,$(1))))

     COMMITCOUNT = $(if $(DUMP),0,$(call commitcount))
    -AUTORELEASE = $(if $(DUMP),0,$(call commitcount,1))
    +AUTORELEASE = $(if $(DUMP),0,$(shell sed -i "s/\$$(AUTORELEASE)/$(call commitcount,1)/" $(CURDIR)/Makefile))

     all:
     FORCE: ;

And this command used to fix affected packages:

    ./scripts/feeds install -a routing
    for i in $(cd feeds/routing; git grep -l 'PKG_RELEASE:=.*AUTORELEASE' | \
                                  sed 's^.*/\([^/]*\)/Makefile^\1^';);
    do
      make package/${i%/Makefile}/download
    done

Signed-off-by: Sven Eckelmann <[email protected]>
* tcpdump: Fix missing sanity check for batman-adv header
* tcpdump: Add missing throughput header length check
* tcpdump: Fix IPv4 header length check
* tcpdump: Add missing ICMPv6 Neighbor Advert length check
* tcpdump: Add missing ICMPv6 Neighbor Solicit length check
* tcpdump: Fix ICMPv4 inner IPv4 header length check

Signed-off-by: Sven Eckelmann <[email protected]>
@ecsv
Copy link
Contributor Author

ecsv commented Feb 3, 2024

Please do no longer set PKG_RELEASE to AUTORELEASE as the feature is deprecated. Please use an integer instead. Below is a list of affected packages including correct PKG_RELEASE:

* batctl/Makefile: PKG_RELEASE:=5

This recommendation is wrong in regards of the PKG_RELEASE value. The old one was basically 3. So the new must be 4 - not 5.

I've added #1043 to once and for all remove AUTORELEASE from the (still not completely EOLed) openwrt-22.03 branch.

@simonwunderlich simonwunderlich merged commit 024b1db into openwrt:openwrt-22.03 Feb 4, 2024
9 of 10 checks passed
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 this pull request may close these issues.

2 participants