Invalid return type for DllCall Topic is solved

[gqqnbig]

Code: Select all

GetModuleFileNameEx( p_pid ) ; by shimanov -  www.autohotkey.com/forum/viewtopic.php?t=9000
{
   local h_process, name_size, result, name

   h_process := DllCall( "OpenProcess", "uint", 0x10|0x400, "int", false, "uint", p_pid )
   if ( ErrorLevel or h_process = 0 )
      return

   name_size:= 255
   VarSetCapacity( name, name_size )

   result := DllCall( "psapi.dll\GetModuleFileNameEx" ( A_IsUnicode ? "W" : "A" )
                 , "uint", h_process, "uint", 0, "str", name, "uint", name_size )

   DllCall "CloseHandle", h_process

   return name
}

This is the script that takes process ID and returns the path of the process. It works fine in AutoHotKey v1 but when it runs in V2, DllCall has error "Invalid return type". How do I fix that?

My AutoHotKey version is a086

无标题.png (12.12 KiB) Viewed 2379 times

[just me]

I doubt that

Code: Select all

DllCall "CloseHandle", h_process

is working in v1.

If you pass an even number of parameters to DllCall(), the last parameter must be the return type (which can be a variable).

[guest3456]

its also weird how the string is getting interpreted as "Close111Handle"

[Helgef]

Sharp eyes guest3456. It doesn't happen on my pc though,

Code: Select all

Error:  Invalid return type.

	Line#
	006: {
	009: h_process := DllCall( "OpenProcess", "uint", 0x10|0x400, "int", false, "uint", p_pid )
	010: if ( ErrorLevel or h_process = 0 )
	011: Return
	013: name_size := 255
	014: VarSetCapacity( name, name_size )
	016: result := DllCall( "psapi.dll\GetModuleFileNameEx" ( A_IsUnicode ? "W" : "A" ), "uint", h_process, "uint", 0, "str", name, "uint", name_size )
--->	019: DllCall("CloseHandle", h_process)
	021: Return name
	022: }
	023: Exit
	024: Exit
	024: Exit

The current thread will exit.

2.0-a088-338ed55

Cheers.

[guest3456]

I doubt that

Code: Select all

DllCall "CloseHandle", h_process

is working in v1.

If you pass an even number of parameters to DllCall(), the last parameter must be the return type (which can be a variable).

yes obviously the code should be

[code]
DllCall "CloseHandle", "uint", h_process
[/code]

Code: Select all

DllCall("CloseHandle", "uint", h_process)

[gqqnbig]

its also weird how the string is getting interpreted as "Close111Handle"

oh, LOL, I was just testing the code