This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[RFA/mi] Fix "-thread-select 123456789" test


Hi,

Now that the mi-pthreads.exp tests are committed, this patch fixes one of 
the failures, which demonstrates that when an error occurs switching 
threads, MI was not able to correctly propagate an error message to the 
user.

This happens because the MI function is using a wrapped gdb function, 
gdb_thread_select. This function returns <0 when an error occurs. The 
wrapped function can return either a gdb failure code (like RETURN_ERROR) 
or a wrapper failures (GDB_RC_FAIL). In both cases, mi_cmd_thread_select 
was returnging MI_CMD_CAUGHT_ERROR, which is only valid in the first case. 
In the latter, ther error message is already in mi_error_message, and it 
should return MI_CMD_ERROR.

You can see this by looking at mi-pthreads.exp's "-thread-select 
123456789" test:

(gdb)
-thread-select 123456789
&"Thread ID 123456789 not known.\n"
^done
(gdb)

Clearly, the correct answer should have ended with '^error,msg="Thread ID 
123456789 not known."'.

Keith

ChangeLog
2002-09-24  Keith Seitz  <keiths@redhat.com>

	* mi-main.c (mi_cmd_thread_select): Only return MI_CMD_CAUGHT_ERROR
	when we really did catch an error(). If we got GDB_RC_FAIL, return
	MI_CMD_ERROR instead.

Patch
Index: mi/mi-main.c
===================================================================
RCS file: /cvs/src/src/gdb/mi/mi-main.c,v
retrieving revision 1.31
diff -p -r1.31 mi-main.c
*** mi/mi-main.c	11 Sep 2002 21:49:04 -0000	1.31
--- mi/mi-main.c	24 Sep 2002 19:44:32 -0000
*************** mi_cmd_thread_select (char *command, cha
*** 244,251 ****
    else
      rc = gdb_thread_select (uiout, argv[0]);
  
!   if (rc == GDB_RC_FAIL)
      return MI_CMD_CAUGHT_ERROR;
    else
      return MI_CMD_DONE;
  }
--- 244,255 ----
    else
      rc = gdb_thread_select (uiout, argv[0]);
  
!   /* RC is enum gdb_rc if it is successful (>=0)
!      enum return_reason if not (<0). */
!   if ((int) rc < 0 && (enum return_reason) rc == RETURN_ERROR)
      return MI_CMD_CAUGHT_ERROR;
+   else if ((int) rc >= 0 && rc == GDB_RC_FAIL)
+     return MI_CMD_ERROR;
    else
      return MI_CMD_DONE;
  }


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]