|
From: Madhu M. K. <mm...@ya...> - 2005-05-24 09:25:17
|
Code path exists for joinee being tracked as NULL to make it here.
--- valgrind.orig/coregrind/vg_threadmodel.c 2005-05-21
00:31:56.000000000 -0700
+++ valgrind/coregrind/vg_threadmodel.c 2005-05-21 01:02:19.000000000 -0700
@@ -449,7 +449,7 @@
break;
}
- if (joinee->detached)
+ if (joinee && joinee->detached)
thread_report(joineeid, THE_Detached, "joining as joiner");
else {
/* block if the joinee hasn't exited yet */
Cheerio,
M
Madhu M Kurup /* Nemo Me Impune Lacessit */ mmk at yahoo-inc dt com
|
|
From: Nicholas N. <nj...@cs...> - 2005-05-25 21:35:03
|
On Tue, 24 May 2005, Madhu M. Kurup wrote: > Code path exists for joinee being tracked as NULL to make it here. > > +++ valgrind/coregrind/vg_threadmodel.c 2005-05-21 01:02:19.000000000 -0700 > @@ -449,7 +449,7 @@ > break; > } > > - if (joinee->detached) > + if (joinee && joinee->detached) > thread_report(joineeid, THE_Detached, "joining as joiner"); Yes, that's a good one, although I don't know if your proposed change is the right way to deal with it. Jeremy? N |