Board index
Microprose's Magic: The Gathering
ManaLink 3.0
Bug Reports
Archived Reports
Pending Reports





[fixed] Bad parameters and CTD.
Moderators: BAgate, drool66, Aswan jaguar, gmzombie, stassy, CCGHQ Admins
[fixed] Bad parameters and CTD.
by etphonehome » 18 Dec 2022, 23:30
Hi
I'm using lstest version.
THX
I'm using lstest version.
A bug!bad parameters
get_card_instance(1, -1)
0: 0x026C1069
1: 0x026B2B43
2: 0x02679F22
3: 0x0230709B
4: 0x023C9785
5: 0x026F0F15
6: 0x026CAFCC
7: 0x026CBB76
8: 0x026CCBE0
9: 0x004722C8
10: 0x00477B2C
11: 0x026AE223
12: 0x0043BFAA
13: 0x004399BD
14: 0x0047902C
15: 0x004946E9
16: 0x761C7BA9
17: 0x77DCBB9B
18: 0x77DCBB1F
THX
- Attachments
-
AUTOSAVE and dump.zip
- (4.52 KiB) Downloaded 121 times
Last edited by drool66 on 21 Dec 2022, 23:47, edited 3 times in total.
Reason: fixed
Reason: fixed
- etphonehome
- Posts: 307
- Joined: 21 May 2020, 12:50
- Has thanked: 281 times
- Been thanked: 46 times
Re: Bad parameters and CTD.
by Aswan jaguar » 20 Dec 2022, 14:40
I tried your savegame 4 times and no error or crash for me.
---
Trying to squash some bugs and playtesting.
Trying to squash some bugs and playtesting.
-
Aswan jaguar - Super Tester Elite
- Posts: 8129
- Joined: 13 May 2010, 12:17
- Has thanked: 748 times
- Been thanked: 477 times
Re: Bad parameters and CTD.
by drool66 » 20 Dec 2022, 16:41
It looks like the -1 card param is from can_legally_play_iid(). The output of add_card_to_hand() needs to be checked before it's used like:
- Code: Select all
int card_added = add_card_to_hand(p, c);
if( card_added != -1 ){
...
The latest images for Manalink will be here.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
-
drool66 - Programmer
- Posts: 1185
- Joined: 25 Nov 2010, 22:38
- Has thanked: 187 times
- Been thanked: 280 times
Re: Bad parameters and CTD.
by drool66 » 21 Dec 2022, 23:45
fixed in 1c9eb63
The latest images for Manalink will be here.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
-
drool66 - Programmer
- Posts: 1185
- Joined: 25 Nov 2010, 22:38
- Has thanked: 187 times
- Been thanked: 280 times
4 posts
• Page 1 of 1
Who is online
Users browsing this forum: No registered users and 2 guests