diff options
| author | Vic (Chun-Ju) Yang <victoryang@chromium.org> | 2014-01-10 12:02:22 +0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-01-11 04:46:50 +0000 |
| commit | 68d005b104e447f647cecec634cea97cef1e6ede (patch) | |
| tree | 0ee16e2fccd4a558aac023ce7e4d60a0422b2574 | |
| parent | dc82ca44d348665132584a21c89ed824562e9696 (diff) | |
| download | chrome-ec-68d005b104e447f647cecec634cea97cef1e6ede.tar.gz | |
Use pthread_equal() to compare thread ID
Pthread document explicitly stated that == operator should not be used
to compare thread IDs. Let's use pthread_equal() to be safe.
BUG=chrome-os-partner:19235
TEST=Check trace dump can be generated from non-main thread
BRANCH=None
Change-Id: I5011e0e380db0ce43c4fcf8fca0d94d5b3108f55
Signed-off-by: Vic (Chun-Ju) Yang <victoryang@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/182039
Reviewed-by: Randall Spangler <rspangler@chromium.org>
| -rw-r--r-- | core/host/task.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/host/task.c b/core/host/task.c index e560b4c59d..08c70da66b 100644 --- a/core/host/task.c +++ b/core/host/task.c @@ -178,7 +178,7 @@ static void __attribute__((noinline)) _task_dump_trace_dispatch(int sig) { int need_dispatch = 1; - if (pthread_self() != main_thread) { + if (!pthread_equal(pthread_self(), main_thread)) { need_dispatch = 0; } else if (!task_start_called()) { fprintf(stderr, "Stack trace of main thread:\n"); |