public inbox for dtrace@lists.linux.dev
 help / color / mirror / Atom feed
* [PATCH] test: Use copyinstr to copy string from user space
@ 2025-10-17  2:21 eugene.loh
  2025-10-17  6:11 ` Kris Van Hees
  0 siblings, 1 reply; 2+ messages in thread
From: eugene.loh @ 2025-10-17  2:21 UTC (permalink / raw)
  To: dtrace, dtrace-devel

From: Eugene Loh <eugene.loh@oracle.com>

Signed-off-by: Eugene Loh <eugene.loh@oracle.com>
---
 test/unittest/proc/tst.exec-execve.sh   | 2 +-
 test/unittest/proc/tst.exec-execveat.sh | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/unittest/proc/tst.exec-execve.sh b/test/unittest/proc/tst.exec-execve.sh
index d3f84e0d2..b70f0a6ea 100755
--- a/test/unittest/proc/tst.exec-execve.sh
+++ b/test/unittest/proc/tst.exec-execve.sh
@@ -42,7 +42,7 @@ proc:::exec
 syscall::execve:entry
 /ppid == dtpid && execname == "parent.x"/
 {
-    printf("execve      %s\n", stringof(arg0));
+    printf("execve      %s\n", copyinstr(arg0));
 }' -c ./parent.x
 if [ $? -ne 0 ]; then
     echo ERROR
diff --git a/test/unittest/proc/tst.exec-execveat.sh b/test/unittest/proc/tst.exec-execveat.sh
index e4b9dfa5e..09a3d9338 100755
--- a/test/unittest/proc/tst.exec-execveat.sh
+++ b/test/unittest/proc/tst.exec-execveat.sh
@@ -44,7 +44,7 @@ proc:::exec
 syscall::execveat:entry
 /ppid == dtpid && execname == "parent.x"/
 {
-    printf("execveat    %s\n", stringof(arg1));
+    printf("execveat    %s\n", copyinstr(arg1));
 }' -c ./parent.x
 if [ $? -ne 0 ]; then
     echo ERROR
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] test: Use copyinstr to copy string from user space
  2025-10-17  2:21 [PATCH] test: Use copyinstr to copy string from user space eugene.loh
@ 2025-10-17  6:11 ` Kris Van Hees
  0 siblings, 0 replies; 2+ messages in thread
From: Kris Van Hees @ 2025-10-17  6:11 UTC (permalink / raw)
  To: eugene.loh; +Cc: dtrace, dtrace-devel

On Thu, Oct 16, 2025 at 10:21:13PM -0400, eugene.loh@oracle.com wrote:
> From: Eugene Loh <eugene.loh@oracle.com>
> 
> Signed-off-by: Eugene Loh <eugene.loh@oracle.com>

Reviewed-by: Kris Van Hees <kris.van.hees@oracle.com>

> ---
>  test/unittest/proc/tst.exec-execve.sh   | 2 +-
>  test/unittest/proc/tst.exec-execveat.sh | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/test/unittest/proc/tst.exec-execve.sh b/test/unittest/proc/tst.exec-execve.sh
> index d3f84e0d2..b70f0a6ea 100755
> --- a/test/unittest/proc/tst.exec-execve.sh
> +++ b/test/unittest/proc/tst.exec-execve.sh
> @@ -42,7 +42,7 @@ proc:::exec
>  syscall::execve:entry
>  /ppid == dtpid && execname == "parent.x"/
>  {
> -    printf("execve      %s\n", stringof(arg0));
> +    printf("execve      %s\n", copyinstr(arg0));
>  }' -c ./parent.x
>  if [ $? -ne 0 ]; then
>      echo ERROR
> diff --git a/test/unittest/proc/tst.exec-execveat.sh b/test/unittest/proc/tst.exec-execveat.sh
> index e4b9dfa5e..09a3d9338 100755
> --- a/test/unittest/proc/tst.exec-execveat.sh
> +++ b/test/unittest/proc/tst.exec-execveat.sh
> @@ -44,7 +44,7 @@ proc:::exec
>  syscall::execveat:entry
>  /ppid == dtpid && execname == "parent.x"/
>  {
> -    printf("execveat    %s\n", stringof(arg1));
> +    printf("execveat    %s\n", copyinstr(arg1));
>  }' -c ./parent.x
>  if [ $? -ne 0 ]; then
>      echo ERROR
> -- 
> 2.47.3
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-10-17  6:11 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-17  2:21 [PATCH] test: Use copyinstr to copy string from user space eugene.loh
2025-10-17  6:11 ` Kris Van Hees

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox