RTEMS | cpukit/libdebugger: Add support for TLS variables (!517)

[Gedare Bloom (@gedare)]

Gedare Bloom started a new discussion on cpukit/libdebugger/rtems-debugger-server.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/517#note_125170

 > +}
 > +
 > +static int
 > +parse_get_tls_addr(
 > +  uint8_t* buffer,
 > +  int size,
 > +  const char **thread_id_str,
 > +  const char **offset_str,
 > +  const char **lm_str
 > +)
 > +{
 > +  if (thread_id_str == NULL || offset_str == NULL || lm_str == NULL) {
 > +    return 1;
 > +  }
 > +
 > +  *thread_id_str = strchr((const char*) buffer, ':') + 1;

if `strchr` fails, this makes `thread_id_str == 1`, not `NULL`, so the next line error check is incorrect

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/517#note_125170
You're receiving this email because of your account on gitlab.rtems.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20250624/8858d06e/attachment-0001.htm>