[PATCH] gdb/testsuite/rocm: fix rocm-multi-inferior-gpu.cpp

[PATCH] gdb/testsuite/rocm: fix rocm-multi-inferior-gpu.cpp

[Lancelot Six]

The gdb/testsuite/gdb.rocm/multi-inferior-gpu.cpp testcase contains a
call to execl which does not have NULL as a last argument.  This is
an invalid use of execl.  This patch fixes this oversight.

Change-Id: I03b60abe30468d71ba5089b240c6d00f9b8883b2
---
 gdb/testsuite/gdb.rocm/multi-inferior-gpu.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gdb/testsuite/gdb.rocm/multi-inferior-gpu.cpp b/gdb/testsuite/gdb.rocm/multi-inferior-gpu.cpp
index ca869233b58..d64afdd1994 100644
--- a/gdb/testsuite/gdb.rocm/multi-inferior-gpu.cpp
+++ b/gdb/testsuite/gdb.rocm/multi-inferior-gpu.cpp
@@ -62,7 +62,7 @@ parent (int argc, char **argv)
       if (pid == 0)
 	{
 	  /* Exec to force the child to re-initialize the ROCm runtime.  */
-	  if (execl (argv[0], argv[0], n) == -1)
+	  if (execl (argv[0], argv[0], n, nullptr) == -1)
 	    {
 	      perror ("Failed to exec");
 	      return -1;

base-commit: cdda72c2fa1e47c566c7b5768f3831a1cc11d263
-- 
2.34.1

Re: [PATCH] gdb/testsuite/rocm: fix rocm-multi-inferior-gpu.cpp

[Tom Tromey]

>>>>> "Lancelot" == Lancelot Six via Gdb-patches <gdb-patches@sourceware.org> writes:

Lancelot> The gdb/testsuite/gdb.rocm/multi-inferior-gpu.cpp testcase contains a
Lancelot> call to execl which does not have NULL as a last argument.  This is
Lancelot> an invalid use of execl.  This patch fixes this oversight.

Thank you.  FAOD and for future reference, I think it could have gone in
by the obvious rule.

Approved-By: Tom Tromey <tom@tromey.com>

Tom

Re: [PATCH] gdb/testsuite/rocm: fix rocm-multi-inferior-gpu.cpp

[Lancelot SIX]

On 11/09/2023 18:52, Tom Tromey wrote:
>>>>>> "Lancelot" == Lancelot Six via Gdb-patches <gdb-patches@sourceware.org> writes:
> 
> Lancelot> The gdb/testsuite/gdb.rocm/multi-inferior-gpu.cpp testcase contains a
> Lancelot> call to execl which does not have NULL as a last argument.  This is
> Lancelot> an invalid use of execl.  This patch fixes this oversight.
> 
> Thank you.  FAOD and for future reference, I think it could have gone in
> by the obvious rule.
> 
> Approved-By: Tom Tromey <tom@tromey.com>
> 
> Tom

Thanks,

I have just pushed this patch.

I did wonder if I should use the obv rule for this, but ended up not 
using it.  I'll keep that in mind for future patches!

Best,
Lancelot.