upnpDiscover method signature has changed and build crashes.

[cdberkstresser]

The upnpDiscover method signature of the miniupnpc package has changed from
upnpDiscover(int delay, const char * multicastif, const char * minissdpdsock, int localport, int ipv6, int * error);
to
upnpDiscover(int delay, const char * multicastif, const char * minissdpdsock, int localport, int ipv6, unsigned char ttl, int * error);
causing the build to crash with MINIUPNPC_VERSION = 2.1. (E.g On Ubuntu 18.10)
I added an unsigned char ttl to the function call to fix it on my machine, but it looks like there may be three method signatures to track now rather than two.

[Serhii-Melnyk]

on the web found patch, tried - looks like i've built it. Hooray!
in /src/net.cpp:1037 "devlist = upnpDiscover(2000, multicastif, minissdpdpath, 0, 0, &error);" added "2" got "devlist = upnpDiscover(2000, multicastif, minissdpdpath, 0, 0, 2, &error);"
used info from https://github.com/bitcoin/bitcoin/commit/9f3e48e5219a09b5ddfd6883d1f0498910eff4b6.patch

it works! (disabled upnp in options, direct connection, no router used)
synchronizing...

[Serhii-Melnyk]

i'm totally not a programmer hope this wont break something :)