llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-lldb Author: aokblast <details> <summary>Changes</summary> These two fields are required by HostTest.cpp. Although I don't see any usecase, we add these two fields to prevent failure. --- Full diff: https://github.com/llvm/llvm-project/pull/209009.diff 1 Files Affected: - (modified) lldb/source/Host/freebsd/Host.cpp (+6) ``````````diff diff --git a/lldb/source/Host/freebsd/Host.cpp b/lldb/source/Host/freebsd/Host.cpp index 1bc0a9a2889af..e9c256d6faabb 100644 --- a/lldb/source/Host/freebsd/Host.cpp +++ b/lldb/source/Host/freebsd/Host.cpp @@ -143,6 +143,8 @@ static bool GetFreeBSDProcessUserAndGroup(ProcessInstanceInfo &process_info) { if (proc_kinfo_size == 0) goto error; + process_info.SetProcessGroupID(proc_kinfo.ki_pgid); + process_info.SetProcessSessionID(proc_kinfo.ki_sid); process_info.SetParentProcessID(proc_kinfo.ki_ppid); process_info.SetUserID(proc_kinfo.ki_ruid); process_info.SetGroupID(proc_kinfo.ki_rgid); @@ -154,6 +156,8 @@ static bool GetFreeBSDProcessUserAndGroup(ProcessInstanceInfo &process_info) { return true; error: + process_info.SetProcessGroupID(LLDB_INVALID_PROCESS_ID); + process_info.SetProcessSessionID(LLDB_INVALID_PROCESS_ID); process_info.SetParentProcessID(LLDB_INVALID_PROCESS_ID); process_info.SetUserID(UINT32_MAX); process_info.SetGroupID(UINT32_MAX); @@ -221,6 +225,8 @@ uint32_t Host::FindProcessesImpl(const ProcessInstanceInfoMatch &match_info, ProcessInstanceInfo process_info; process_info.SetProcessID(kinfo.ki_pid); + process_info.SetProcessGroupID(kinfo.ki_pgid); + process_info.SetProcessSessionID(kinfo.ki_sid); process_info.SetParentProcessID(kinfo.ki_ppid); process_info.SetUserID(kinfo.ki_ruid); process_info.SetGroupID(kinfo.ki_rgid); `````````` </details> https://github.com/llvm/llvm-project/pull/209009 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
