[RFC] fix win32-nat failure

[Pierre Muller]

  Current GDB head has a problem with win32-nat:

When starting ./gdb ./gdb
(gdb) run
Starting program: /usr/local/src/gdbcvs/build-bare/gdb/gdb.exe ./gdb
../../purecvs/gdb/inferior.c:41: internal-error: current_inferior: Assertion
`inf' failed.



This comes from a call to
Breakpoint 1, internal_error (file=0x6525d0 "../../purecvs/gdb/inferior.c",
    line=41, string=0x6525b1 "%s: Assertion `%s' failed.")
    at ../../purecvs/gdb/utils.c:897
897       va_start (ap, string);
(top-gdb) bt
#0  internal_error (file=0x6525d0 "../../purecvs/gdb/inferior.c", line=41,
    string=0x6525b1 "%s: Assertion `%s' failed.")
    at ../../purecvs/gdb/utils.c:897
#1  0x0043d116 in current_inferior () at ../../purecvs/gdb/inferior.c:41
#2  0x0042e1bf in terminal_inferior () at ../../purecvs/gdb/inflow.c:280
During symbol reading, struct/union type gets multiply defined: struct
language_
defn.
#3  0x004753bc in do_initial_win32_stuff (pid=4996, attaching=0)

I was able to fix this by setting inferior_ptid before terminal_inferior was
called,
but I did this using also the ThreadId available when starting with
CreateProcess.
  But there is no thread number available for processes to which we attach,
thus I simply passed zero in that case...

  I don't know, maybe the correct fix is to only set inferior_ptid
to ptid_build(pid, 0, 0)

  Anyhow, it would be nice if this could be fixed ASAP.


Pierre Muller
Pascal language support maintainer for GDB






Index: gdb/win32-nat.c
===================================================================
RCS file: /cvs/src/src/gdb/win32-nat.c,v
retrieving revision 1.164
diff -u -p -r1.164 win32-nat.c
--- gdb/win32-nat.c     22 Sep 2008 15:21:30 -0000      1.164
+++ gdb/win32-nat.c     1 Oct 2008 12:10:33 -0000
@@ -1521,7 +1521,7 @@ win32_wait (ptid_t ptid, struct target_w
 }

 static void
-do_initial_win32_stuff (DWORD pid, int attaching)
+do_initial_win32_stuff (DWORD pid, DWORD tid, int attaching)
 {
   extern int stop_after_trap;
   int i;
@@ -1548,6 +1548,7 @@ do_initial_win32_stuff (DWORD pid, int a
   init_wait_for_inferior ();

   inf = add_inferior (pid);
+  inferior_ptid = ptid_build (pid, 0, tid);
   inf->attach_flag = attaching;

   terminal_init_inferior_with_pgrp (pid);
@@ -1729,7 +1730,7 @@ win32_attach (char *args, int from_tty)
       gdb_flush (gdb_stdout);
     }

-  do_initial_win32_stuff (pid, 1);
+  do_initial_win32_stuff (pid, 0, 1);
   target_terminal_ours ();
 }

@@ -1939,7 +1940,7 @@ win32_create_inferior (char *exec_file,
   else
     saw_create = 0;

-  do_initial_win32_stuff (pi.dwProcessId, 0);
+  do_initial_win32_stuff (pi.dwProcessId, pi.dwThreadId, 0);

   /* win32_continue (DBG_CONTINUE, -1); */
 }