https://github.com/labath commented:

The field this is consuming is actually 17 bytes long, because the process name 
is in parenthesis. I suspect this will cause the function to reject any process 
whose name is longer than 13 characters.

The name field is actually quite hard to parse this way since it can contain 
any character (esp. parenthesis and spaces). Now, we could devise an algorithm 
to do that, but since the code is later opening `/proc/$PID/status` anyway, and 
`status` contains a superset of information, I think it'd be best to just 
delete this code and extract the information we want from there.

`status` parsing code also uses more modern and less error prone patterns.

https://github.com/llvm/llvm-project/pull/94783
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to