Re: server: Change the get_dll_info server request to allow retrievingthe image file name of a process.

[Dmitry Timoshkov]

"Robert Shearman"  wrote:
> -
>    if (req->exe_module)
> -
>        dll = list_head( &process->dlls ) ?
> -
>            LIST_ENTRY(list_head( &process->dlls ), struct process_dll, entry) : NULL;
> -
>    else
> -
>        dll = find_process_dll( process, req->base_address );

Is it really necessary to introduce the exe_module request parameter?
Main executable has its own base address which should work just fine
in find_process_dll().

server: Change the get_dll_info server request to allow retrievingthe image file name of a process.

[Robert Shearman]

Dmitry Timoshkov wrote:
> "Robert Shearman"  wrote:
> > -
> >    if (req->exe_module)
> > -
> >        dll = list_head( &process->dlls ) ?
> > -
> >            LIST_ENTRY(list_head( &process->dlls ), struct 
> >
> > process_dll, entry) : NULL;
> > -
> >    else
> > -
> >        dll = find_process_dll( process, req->base_address );
>
> Is it really necessary to introduce the exe_module request parameter?

Yes.
> Main executable has its own base address which should work just fine
> in find_process_dll().

Really? http://msdn2.microsoft.com/en-us/library/f7f5138s(VS.71).aspx

server: Change the get_dll_info server request to allow retrievingthe image file name of a process.

[Alexandre Julliard]

Robert Shearman  writes:
> Dmitry Timoshkov wrote:
> > Is it really necessary to introduce the exe_module request parameter?
>
> Yes.
> > Main executable has its own base address which should work just fine
> > in find_process_dll().
>
> Really? http://msdn2.microsoft.com/en-us/library/f7f5138s(VS.71).aspx

The usual convention is that a NULL module address means the main exe, I
think that would be appropriate here.

server: Change the get_dll_info server request to allowretrievingthe image file name of a process.

[Dmitry Timoshkov]

"Robert Shearman"  wrote:
> > Main executable has its own base address which should work just fine
> > in find_process_dll().
>
> Really? http://msdn2.microsoft.com/en-us/library/f7f5138s(VS.71).aspx

I don't see how the page above is related to what I said. Main exe module
is being added to the process dlls list at the end of the initialization
process by load_dll server call with its real base address.

server: Change the get_dll_info server request to allowretrievingthe image file name of a process.

[Robert Shearman]

Dmitry Timoshkov wrote:
> "Robert Shearman"  wrote:
> > > Main executable has its own base address which should work just fine
> > > in find_process_dll().
> >
> > Really? http://msdn2.microsoft.com/en-us/library/f7f5138s(VS.71).aspx
>
> I don't see how the page above is related to what I said. Main exe module
> is being added to the process dlls list at the end of the initialization
> process by load_dll server call with its real base address.

I don't see how that is relevant either. We don't know the main exe 
module base address in anything other than the server and that one process.