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

UPNP build error #101

Closed
ogarcia opened this issue Jul 1, 2024 · 1 comment · Fixed by #109
Closed

UPNP build error #101

ogarcia opened this issue Jul 1, 2024 · 1 comment · Fixed by #109

Comments

@ogarcia
Copy link

ogarcia commented Jul 1, 2024

Arch Linux with libnatpmp-20230423-2 and miniupnpc-2.2.8-1:

src/utils/net/upnp_stubs.c: In function ‘upnpPulse’:
src/utils/net/upnp_stubs.c:659:41: error: too few arguments to function ‘UPNP_GetValidIGD’
  659 |         if( UPNP_IGD_VALID_CONNECTED == UPNP_GetValidIGD( devlist, &map->upnpUrls, &map->upnpData,
      |                                         ^~~~~~~~~~~~~~~~

This problem occurred after upgrading miniupnpc from 2.2.7-2 to 2.2.8-1 so the culprit is this update.

@FabioLolix
Copy link

FabioLolix commented Jul 8, 2024

In the Arch AUR pkgbuild the problem is avoided using

sed -i 's/sizeof( map->lanaddr )/sizeof( map->lanaddr ), NULL , 0/g' src/utils/net/upnp_stubs.c

as suggested here

barracuda156 added a commit to barracuda156/mldonkey that referenced this issue Aug 15, 2024
@ygrek ygrek closed this as completed in a22b318 Aug 17, 2024
carlonluca pushed a commit to carlonluca/mldonkey that referenced this issue Aug 17, 2024
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.

2 participants