From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6582 invoked by alias); 20 Feb 2002 16:41:08 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 6463 invoked from network); 20 Feb 2002 16:41:04 -0000 Received: from unknown (HELO cerbere.u-strasbg.fr) (130.79.112.7) by sources.redhat.com with SMTP; 20 Feb 2002 16:41:04 -0000 Received: from laocoon (laocoon.u-strasbg.fr [130.79.112.72]) by cerbere.u-strasbg.fr (8.9.3/8.8.7) with ESMTP id RAA25919 for ; Wed, 20 Feb 2002 17:41:02 +0100 Message-Id: <4.2.0.58.20020220171640.0168c2b0@ics.u-strasbg.fr> X-Sender: muller@ics.u-strasbg.fr X-Mailer: QUALCOMM Windows Eudora Pro Version 4.2.0.58 Date: Wed, 20 Feb 2002 08:41:00 -0000 To: gdb-patches@sources.redhat.com From: Pierre Muller Subject: Re: [PATCH] run > file for win32 In-Reply-To: <20020220161113.GH26603@redhat.com> References: <4.2.0.58.20020220101817.00ace9a0@ics.u-strasbg.fr> <20020216023944.GA22542@redhat.com> <4.2.0.58.20020220101817.00ace9a0@ics.u-strasbg.fr> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" X-SW-Source: 2002-02/txt/msg00542.txt.bz2 > >To sort the events and only handles those of the process that you want > >to debug you use saw_create variable. > >IMO, the right way to handle this is to inspect the name of each process >as it is created. That presents some interesting difficulties, however. >It means that gdb has to understand how the shell will find the executable, >however. > >Alternatively, gdb could detect the transition from shell -> program where >the pid stays the same. Hmm. I think that would be the most foolproof. But that is not the case... the windows pid does change when you pass from the shell to the debuggee. My patch is even based on that change. >I'll try to come up with some kind of solution to this that uses your method >plus the above. It will probably take a couple of days, though. As the win32 API clearly says that the lpImageName field of the CREATE_PROCESS_DEBUG_EVENT can be null, this is not an option. May I suggest using GetFileInformationByHandle function? If you open the file in the child_create_inferior function and leave it open until the program is killed or exits, then the nFileIndexHigh,nFileIndexLow and dwVolumeSerialNumber identifies uniquely the file. Using the handle given by the CREATE_PROCESS_DEBUG_EVENT and calling again GetFileInformationByHandle should be able to check if the correct file is used. Remarks: -- if a different file (in some other dir) is found, it will need to be handled somehow... -- Even with my patch, there is still a problem: the shell does not exit (because there is one more ContinueDebugEvent needed to the shell exit also, that is at least the feeling I had...) So after runnig the same program 10 times, you have got 10 zomby shells. -- I don't know if this is a problem with my configuration, but the getenv("SHELL") returns NULL in _initialize_inftarg while I do have SHELL=/bin/bash in the bash that I use as shell... -- This reminds me of one more question: do all shells accept "exec" command and -c option ? -- We need to be very careful about the different handles that I given by the different events, failing to close some might result in problems later. Finally, wouldn't it be wise to set the default value of useshell to 0 until this problem is fixed? Currently the variable is set to 1 in _initialize_inftarg if a shell is found. Pierre Muller Institut Charles Sadron 6,rue Boussingault F 67083 STRASBOURG CEDEX (France) mailto:muller@ics.u-strasbg.fr Phone : (33)-3-88-41-40-07 Fax : (33)-3-88-41-40-99