Program-assigned thread names on Windows
Jon Turney
jon.turney@dronecode.org.uk
Wed Jul 27 21:35:00 GMT 2016
Doing a bit of testing with this patch...
On 26/07/2016 18:15, LRN wrote:
> + named_thread = find_thread_ptid (named_thread_ptid);
... it seems this may return NULL (e.g. if the thread we are naming was
short-lived and has already exited, or if the thread id was invalid) ...
> +
> + thread_name = NULL;
> + thread_name_len = target_read_string (thread_name_target, &thread_name, 1025, 0);
> + if (thread_name_len > 0 && thread_name != NULL)
> + {
> + thread_name[thread_name_len - 1] = '\0';
> + if (thread_name[0] != '\0')
> + {
> + xfree (named_thread->name);
... so this becomes a null dereference.
> + named_thread->name = thread_name;
> + }
> + else
> + {
> + xfree (thread_name);
> + }
More information about the Gdb-patches
mailing list