Page 1 of 2

Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 1:32 pm
by romaniac
Ok, this time I'll start the thread. After today's patch, bot stops with an error message during startup:

Code: Select all

.../rom/classes/memorytable.lua:7: bad argument
 #2 to 'memoryReadIntPtr' (number expected, got nil)
A previous call of update showed a lot of output without warnings.

What did they change this time? :-)

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 2:11 pm
by botique
my update ends with:
"...grams/micromacro/scripts/rom/classes/memorytable.lua:7: bad argument #2 to 'memoryReadIntPtr' (number expected, got nil)"

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 2:23 pm
by Sithlord512589
my update also ends with

Code: Select all

.../rom/classes/memorytable.lua:7: bad argument
 #2 to 'memoryReadIntPtr' (number expected, got nil)

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 3:15 pm
by gloover
ARRRGGH - this sucks!
Your quest is this: Be patient!

stay cool and wait - our experts (admin, lisa, rock) will fix this asap!

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 3:56 pm
by _hirondelle
Nice Gloover ;)

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 4:02 pm
by SpiralV
edit addresses.lua
tablesBase = 0x9FF4C4,
Thanks to Lisa for her howto ^^
http://www.solarstrike.net/phpBB3/viewt ... 490#p24490

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 4:07 pm
by BillDoorNZ
tablesBase = 0x9FF4C8,

gets it going...after y've done an update.lua that is :)

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 4:18 pm
by Ego95
So which adress is the right one? :D
0x9FF4C4 or 0x9FF4C8? :D

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 4:21 pm
by BillDoorNZ
lol...surely you can just try both???

but:

tablesBase = 0x9FF4C8,

is the right one

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 4:26 pm
by Ego95
Thank you, then I can turn on my computer and bot tonight :)
You're right, why do i ask when I can try both in less than 1 minute -.-
I need some sleep...

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 4:29 pm
by SpiralV
BillDoorNZ wrote:lol...surely you can just try both???

but:

tablesBase = 0x9FF4C8,

is the right one
Both are correct, so the first one is mine and the second one yours :lol:

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 4:32 pm
by BillDoorNZ
how interesting :)

I just modify the memorytables.lua file to change the beginning to:

Code: Select all

	local tablePointer;
	local startAddressOffsets;
	local dataPointer;

	for i=0,(4*100),4 do
		tablePointer = memoryReadIntPtr( getProc(), addresses.tablesBase+(i), addresses.tablesBaseOffset )
		if ((tablePointer) and ((tablePointer or 0) > 10000)) then
			printf("-"..i..": "..tablePointer.."\n");
			startAddressOffsets = {0,addresses.tableStartPtrOffset, addresses.tableDataStartPtrOffset}
			dataPointer = memoryReadIntPtr( getProc(), tablePointer , startAddressOffsets );
			
			if (dataPointer) then
				printf("-"..i..": "..tablePointer.." - "..dataPointer.."\n");
				dataPointer = dataPointer - lineSize;
				break;
			end;
		end;
	end;
so it cycles through the current base address adding 4 each time (int size). Or I change the + to a - to go back other way and HOPE ;)

I think yours will be more correct if you did it with Lisa's method

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 4:37 pm
by romaniac
BillDoorNZ wrote: tablesBase = 0x9FF4C8,

is the right one
Confirmed. Works nicely. Thank you.

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 5:26 pm
by Administrator
I'm still downloading the client (it is god-awful slow), so I'm not able to test anything just yet. Still, I've made a SVN commit with the address change you guys have reported working. If there's anything else, just post it in this thread.

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 5:30 pm
by gloover
Nope, all adresses could be updated with the bot - everything seems to work.

Nice work.

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 7:05 pm
by SpiralV
Hmmm,
the SVN addresses.lua is not OK, looks like there is no auto-update has been made.
I'll post mine, changes to previous version:
- run update.lua
-tablesBase = 0x9FF4C4, (got a problem with 0x9FF4C8 on a different PC)
-partyMemberList_address = 0x9FE238,

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 7:20 pm
by lisa
SpiralV wrote:Hmmm,
the SVN addresses.lua is not OK, looks like there is no auto-update has been made.
I'll post mine, changes to previous version:
- run update.lua
-tablesBase = 0x9FF4C4, (got a problem with 0x9FF4C8 on a different PC)
-partyMemberList_address = 0x9FE238,
Admin would have just added in the tablebase address, so people would still need to do a rom/update to update the rest of the addresses.

I'll commit with this addresses.lua but yeah I haven't updated RoM yet either so I can't test them.
rev 730

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 8:21 pm
by SpiralV
Ah sry, i guessed this value would be overwritten during an update.

Re: Adjustments for this weeks patch needed.

Posted: Tue Jul 24, 2012 8:22 pm
by lisa
If anyone is interested I use this userfunction to test if the addresses have updated correctly.

For best results be in a party and also the party leader, also have a eggpet in first eggpet slot.

Just run commandline

Code: Select all

rom/bot path:commandline
and type

Code: Select all

testaddresses()
You will get prints on MM screen, it will also use the fly hack and ask if you are flying, answer yes or no
userfunction_testaddresses.lua
(2.55 KiB) Downloaded 267 times
It will look something like this

Code: Select all

Command> testaddresses()
staticbase_char: Confirmed
staticInventory: Confirmed
moneyPtr: Confirmed
eggPetBaseAddress: No pet to test with
zoneId: Confirmed
partyMemberList_address: Unconfirmed as not in a party
Swimhack ACTIVATED!
Am I flying??
>>yes
swimAddress: Confirmed
Swimhack DEactivated.
Command>
In this test I had no eggpet and wasn't in a party.

Re: Adjustments for this weeks patch needed.

Posted: Wed Jul 25, 2012 12:39 am
by gloover
Hey lisa,

nice idea - will test your userfunction. I have take a look inside: does this also support warlock and champ?
Couldnt see those classes listed inside.