[Bug 1912783] Please test proposed package

[Timo Aaltonen]

Hello Kleber, or anyone else affected,

Accepted west-chamber into groovy-proposed. The package will build now
and be available at https://launchpad.net/ubuntu/+source/west-
chamber/20100405+svn20111107.r124-12ubuntu0.3 in a few hours, and then
in the -proposed repository.

Please help us by testing this new package.  See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how
to enable and use -proposed.  Your feedback will aid us getting this
update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug,
mentioning the version of the package you tested, what testing has been
performed on the package and change the tag from verification-needed-
groovy to verification-done-groovy. If it does not fix the bug for you,
please add a comment stating that, and change the tag to verification-
failed-groovy. In either case, without details of your testing we will
not be able to proceed.

Further information regarding the verification process can be found at
https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  Thank you in
advance for helping!

N.B. The updated package will be released to -updates after the bug(s)
fixed by this package have been verified and the package has been in
-proposed for a minimum of 7 days.

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors Team, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1912783

Title:
  west-chamber-dkms fails to build after ip_route_me_harder() signature
  change

Status in west-chamber package in Ubuntu:
  Fix Released
Status in west-chamber source package in Bionic:
  Fix Committed
Status in west-chamber source package in Focal:
  Fix Committed
Status in west-chamber source package in Groovy:
  Fix Committed

Bug description:
  [Impact]
  Focal linux 5.4.0-57.63 and Groovy linux 5.8.0-44.50 backported from upstream stable releases the following commit, which broke the build of west-chamber dkms modules on Groovy, Focal and Bionic for all 5.4 and 5.8 kernels based on this release:

  "netfilter: use actual socket sk rather than skb sk when routing
  harder" (Upstream commit 46d6c5ae953cc0be38efd0e469284df7c4328cf8).

  This is the same issue found on xtables-addons (bug 1907109 and bug
  1915177).

  [Test case]
  Install west-chamber-dkms package which builds the kernel modules.

  [Fix]
  The proposed fix is similar to what has been done for xtabled-addons and use a version of the API depending on the kernel version being built against.

  However, the ABI version detection needs to be done differently
  between the 5.4 and the 5.8 kernels. For the 5.4 kernels the ABI
  change was applied as part of an upstream stable release which changed
  the SUBLEVEL version on the Makefile, so the detection can be done
  using the LINUX_VERSION_CODE macro. For the 5.8 kernels, this can't be
  done and another method is needed to check function declaration on the
  kernel headers. Therefore Focal also needs the same change as done in
  Groovy because of the HWE backport based on 5.8.

  [Where problems could occur]
  If any kernel version doesn't match the assumptions reflected by the ifdef's the modules can fail to build.

  -----
  Testing failing on:
  amd64: https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-focal/focal/amd64/w/west-chamber/20210121_172911_d9f41@/log.gz
  arm64: https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-focal/focal/arm64/w/west-chamber/20210113_011159_54cdf@/log.gz
  armhf: https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-focal/focal/armhf/w/west-chamber/20210113_010911_54cdf@/log.gz
  ppc64el: https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-focal/focal/ppc64el/w/west-chamber/20210113_010547_54cdf@/log.gz
  s390x: https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-focal/focal/s390x/w/west-chamber/20210113_010347_cc516@/log.gz

  Build of the dkms is failing with the following error:

  /usr/src/west-chamber-20100405+svn20111107.r124/extensions/xt_CUI.c:106:30: error: passing argument 2 of ‘ip_route_me_harder’ from incompatible pointer type [-Werror=incompatible-pointer-types]
    106 |  if (ip_route_me_harder(net, skb, addr_type))
        |                              ^~~
        |                              |
        |                              struct sk_buff *
  In file included from ./include/linux/netfilter/x_tables.h:245,
                   from /usr/src/west-chamber-20100405+svn20111107.r124/extensions/xt_CUI.c:19:
  ./include/linux/netfilter_ipv4.h:19:54: note: expected ‘struct sock *’ but argument is of type ‘struct sk_buff *’
     19 | int ip_route_me_harder(struct net *net, struct sock *sk, struct sk_buff *skb, unsigned addr_type);
        |

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/west-chamber/+bug/1912783/+subscriptions