Using enet causes Cannot push enet peer to Lua: pointer value %p is too large

[kungfoo]

I tried updating my app to the newest main branch state and it is using enet for communication with the server.

When I try to connect the enet host, the app throws this exception. This is with a debug build on a Pixel 4 (updated to Android 12).

Not sure if I'm building it wrong, or if there is something wrong.

[MikuAuahDark]

Looks like the fix is trivial. love2d/love@4f491c2 (#200) just needs to be applied to ENet too.

[slime73]

Just to confirm, your local code has the change AuahDark linked?

[kungfoo]

I deleted the invalid comment: nogame.lua works fine, using enet is what crashes it.

[kungfoo]

I ported a really ugly hacky version of the fix and can report that it works. I can try preparing a PR for upstream, if desired.

[MikuAuahDark]

Sure. I was bit puzzled since I don't know the obvious location where actually those keys are used.

[slime73]

I think it's just compute_peer_key / push_peer (in enet.cpp) that need to change, similar to the main LÖVE commit. I'll see about implementing it shortly.

[kungfoo]

That's what i did. I just ported the methods to the types `EnetPeer`. I'm pretty sure there is a nicer way to do it, though.

…

On Mon, Dec 13, 2021, 04:28 Alex Szpakowski ***@***.***> wrote: I think it's just compute_peer_key / push_peer (in enet.cpp) that need to change, similar to the main LÖVE commit. I'll see about implementing it shortly. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#222 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAKAKHL5HPCSZMNEWQAMHTUQVR53ANCNFSM5JWJMQVQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.