Project

General

Profile

Actions

Bug #13829

closed

NUL char in $0

Added by nobu (Nobuyoshi Nakada) over 6 years ago. Updated over 6 years ago.

Status:
Closed
Assignee:
-
Target version:
-
[ruby-core:82425]

Description

$0 and Process.setproctitle ignore \0 and the after.
Is it intentional?

$ ruby -e '$0 = "foo\0bar"; system("ps", "#$$"); Process.setproctitle("bar\0foo"); system("ps", "#$$")'
  PID   TT  STAT      TIME COMMAND
34931 s001  S+     0:00.06 foo  
  PID   TT  STAT      TIME COMMAND
34931 s001  S+     0:00.06 bar  

I'd expect an exception instead.

commit c136175ff118c7837948cf6fbdcdd0d96bc8a837
Author: Nobuyoshi Nakada <nobu@ruby-lang.org>
Date:   Fri Aug 18 22:30:06 2017

    ruby.c: reject NUL in $0
    
    * ruby.c (ruby_setproctitle): raise if the argument contains NUL
      char.  process title is a NUL-terminated string.

diff --git a/ruby.c b/ruby.c
index f2f3c86e09..09ab1c24c4 100644
--- a/ruby.c
+++ b/ruby.c
@@ -2041,6 +2041,8 @@ proc_argv0(VALUE process)
     return rb_orig_progname;
 }
 
+static VALUE ruby_setproctitle(VALUE title);
+
 /*
  *  call-seq:
  *     Process.setproctitle(string)  -> string
@@ -2061,10 +2063,14 @@ proc_argv0(VALUE process)
 static VALUE
 proc_setproctitle(VALUE process, VALUE title)
 {
-    StringValue(title);
-
-    setproctitle("%.*s", RSTRING_LENINT(title), RSTRING_PTR(title));
+    return ruby_setproctitle(title);
+}
 
+static VALUE
+ruby_setproctitle(VALUE title)
+{
+    const char *ptr = StringValueCStr(title);
+    setproctitle("%.*s", RSTRING_LENINT(title), ptr);
     return title;
 }
 
@@ -2074,7 +2080,7 @@ set_arg0(VALUE val, ID id)
     if (origarg.argv == 0)
 	rb_raise(rb_eRuntimeError, "$0 not initialized");
 
-    rb_progname = rb_str_new_frozen(proc_setproctitle(rb_mProcess, val));
+    rb_progname = rb_str_new_frozen(ruby_setproctitle(val));
 }
 
 static inline VALUE
diff --git a/test/ruby/test_rubyoptions.rb b/test/ruby/test_rubyoptions.rb
index d6297c8979..5d2967e65e 100644
--- a/test/ruby/test_rubyoptions.rb
+++ b/test/ruby/test_rubyoptions.rb
@@ -534,6 +534,13 @@
   def test_setproctitle
     skip "platform dependent feature" unless defined?(PSCMD) and PSCMD
 
+    assert_separately([], "#{<<-"begin;"}\n#{<<-'end;'}")
+    begin;
+      assert_raise(ArgumentError) do
+        Process.setproctitle("hello\0")
+      end
+    end;
+
     with_tmpchdir do
       write_file("test-script", "$_0 = $0.dup; Process.setproctitle('hello world'); $0 == $_0 or Process.setproctitle('$0 changed!'); sleep 60")
 
Actions #1

Updated by nobu (Nobuyoshi Nakada) over 6 years ago

  • Status changed from Open to Closed

Applied in changeset trunk|r60215.


ruby.c: reject NUL in $0

  • ruby.c (ruby_setproctitle): raise if the argument contains NUL
    char. process title is a NUL-terminated string.
    [ruby-core:82425] [Bug #13829]
Actions

Also available in: Atom PDF

Like0
Like0