morgan/Grinch-AP
1
0
Fork 0
mirror of https://github.com/MarioSpore/Grinch-AP.git synced 2025-10-21 20:21:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

SNIClient/LttP: modern SNI prevents payload overflow (#2523)

Browse Source
This commit is contained in:
Fabian Dill
2024-04-14 20:40:09 +02:00
committed by GitHub
parent 842a15fd3c
commit fb1cf26118
2 changed files with 8 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
SNIClient.py
View File

@@ -564,14 +564,10 @@ async def snes_write(ctx: SNIContext, write_list: typing.List[typing.Tuple[int,
PutAddress_Request: SNESRequest = {"Opcode": "PutAddress", "Operands": [], 'Space': 'SNES'} PutAddress_Request: SNESRequest = {"Opcode": "PutAddress", "Operands": [], 'Space': 'SNES'}
try: try:
for address, data in write_list: for address, data in write_list:
while data:
# Divide the write into packets of 256 bytes.
PutAddress_Request['Operands'] = [hex(address)[2:], hex(min(len(data), 256))[2:]] PutAddress_Request['Operands'] = [hex(address)[2:], hex(min(len(data), 256))[2:]]
if ctx.snes_socket is not None: if ctx.snes_socket is not None:
await ctx.snes_socket.send(dumps(PutAddress_Request)) await ctx.snes_socket.send(dumps(PutAddress_Request))
await ctx.snes_socket.send(data[:256]) await ctx.snes_socket.send(data)
address += 256
data = data[256:]
else: else:
snes_logger.warning(f"Could not send data to SNES: {data}") snes_logger.warning(f"Could not send data to SNES: {data}")
except ConnectionClosed: except ConnectionClosed:

5
worlds/alttp/Options.py
View File

@@ -716,9 +716,8 @@ class BeemizerTrapChance(BeemizerRange):
display_name = "Beemizer Trap Chance" display_name = "Beemizer Trap Chance"
class AllowCollect(Toggle): class AllowCollect(DefaultOnToggle):
"""Allows for !collect / co-op to auto-open chests containing items for other players. """Allows for !collect / co-op to auto-open chests containing items for other players."""
Off by default, because it currently crashes on real hardware."""
display_name = "Allow Collection of checks for other players" display_name = "Allow Collection of checks for other players"