diff options
| -rw-r--r-- | nptl_db/ChangeLog | 9 | ||||
| -rw-r--r-- | nptl_db/td_ta_event_getmsg.c | 27 |
2 files changed, 29 insertions, 7 deletions
diff --git a/nptl_db/ChangeLog b/nptl_db/ChangeLog index 86f1dc7..cb5b015 100644 --- a/nptl_db/ChangeLog +++ b/nptl_db/ChangeLog @@ -1,3 +1,12 @@ +2003-03-18 Roland McGrath <roland@redhat.com> + + * td_thr_event_getmsg.c (td_thr_event_getmsg): Splice the thread out + of the ->nextevent linkage. + + * td_ta_event_getmsg.c (td_ta_event_getmsg): Runtime error instead of + assert for reading TD_EVENT_NONE. Clear the event buffer after + reading it. Add a sanity check for foo->nextevent = foo. + 2003-03-15 Roland McGrath <roland@redhat.com> * thread_db.h (td_err_e): Add TD_NOTLS and TD_TLSDEFER. diff --git a/nptl_db/td_ta_event_getmsg.c b/nptl_db/td_ta_event_getmsg.c index ae3f8e7..bab44cd 100644 --- a/nptl_db/td_ta_event_getmsg.c +++ b/nptl_db/td_ta_event_getmsg.c @@ -18,7 +18,6 @@ Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA. */ -#include <assert.h> #include <stddef.h> #include <string.h> @@ -55,7 +54,8 @@ td_ta_event_getmsg (const td_thragent_t *ta, td_event_msg_t *msg) return TD_ERR; /* XXX Other error value? */ /* If the structure is on the list there better be an event recorded. */ - assert (event.eventnum != TD_EVENT_NONE); + if (event.eventnum == TD_EVENT_NONE) + return TD_DBERR; /* Generate the thread descriptor. */ th.th_ta_p = (td_thragent_t *) ta; @@ -66,22 +66,35 @@ td_ta_event_getmsg (const td_thragent_t *ta, td_event_msg_t *msg) msg->th_p = &th; msg->msg.data = (uintptr_t) event.eventdata; + /* And clear the event message in the target. */ + memset (&event, '\0', sizeof (td_eventbuf_t)); + if (ps_pdwrite (ta->ph, (char *) addr + offsetof (struct pthread, eventbuf), + &event, sizeof (td_eventbuf_t)) != PS_OK) + return TD_ERR; /* XXX Other error value? */ + /* Get the pointer to the next descriptor with an event. */ psaddr_t next; if (ps_pdread (ta->ph, (char *) addr + offsetof (struct pthread, nextevent), &next, sizeof (struct pthread *)) != PS_OK) return TD_ERR; /* XXX Other error value? */ + if (next == addr) + return TD_DBERR; + /* Store the pointer in the list head variable. */ if (ps_pdwrite (ta->ph, ta->pthread_last_event, &next, sizeof (struct pthread *)) != PS_OK) return TD_ERR; /* XXX Other error value? */ - /* Clear the next pointer in the current descriptor. */ - next = NULL; - if (ps_pdwrite (ta->ph, (char *) addr + offsetof (struct pthread, nextevent), - &next, sizeof (struct pthread *)) != PS_OK) - return TD_ERR; /* XXX Other error value? */ + if (next != NULL) + { + /* Clear the next pointer in the current descriptor. */ + next = NULL; + if (ps_pdwrite (ta->ph, + (char *) addr + offsetof (struct pthread, nextevent), + &next, sizeof (struct pthread *)) != PS_OK) + return TD_ERR; /* XXX Other error value? */ + } return TD_OK; } |