Project

General

Profile

Actions

Bug #214

closed

$0に長い文字列を入れるとpsの出力に環境変数らしきものが出てくる

Added by takeru (takeru sasaki) over 16 years ago. Updated over 13 years ago.

Status:
Closed
Assignee:
-
Target version:
-
ruby -v:
[ruby-dev:35308]

Description

=begin
$0に長い文字列を入れるとpsの出力に環境変数らしきものが出てきます。

Macで発生しますが、Linuxでは発生しませんでした。
macでruby1.9でも発生しませんでした。

ログを添付します。

uname -a
Darwin tkrmb.local 8.11.1 Darwin Kernel Version 8.11.1: Wed Oct 10 18:23:28 PDT 2007; root:xnu-792.25.20~1/RELEASE_I386 i386 i386
=end


Files

zero.txt (1.66 KB) zero.txt takeru (takeru sasaki), 07/02/2008 02:12 PM
Actions #1

Updated by znz (Kazuhiro NISHIYAMA) over 16 years ago

=begin
西山和広です。

At Wed, 2 Jul 2008 14:12:14 +0900,
takeru sasaki wrote:

$0に長い文字列を入れるとpsの出力に環境変数らしきものが出てきます。

Macで発生しますが、Linuxでは発生しませんでした。
macでruby1.9でも発生しませんでした。

IRCでなかむら(う)さんが作成したパッチをちょっと直した以下のパッチで
直るようです。

1.9の方はruby.c:ruby_sysinitでmallocしたものがorigarg.argvに
入っていて、本当のargvが書き換わっていないようです。

Index: ruby.c

--- ruby.c (リビジョン 17840)
+++ ruby.c (作業コピー)
@@ -1039,6 +1039,39 @@ set_arg0space()
#define set_arg0space() ((void)0)
#endif

+static int
+get_arglen(int argc, char **argv)
+{

  • char *s = argv[0];
  • int i;
  • if (!argc) return 0;
  • s += strlen(s);
  • /* See if all the arguments are contiguous in memory */
  • for (i = 1; i < argc; i++) {
  • if (argv[i] == s + 1) {
  •  s++;
    
  •  s += strlen(s);	/* this one is ok too */
    
  • }
  • else {
  •  break;
    
  • }
  • }
    +#if defined(USE_ENVSPACE_FOR_ARG0)
  • if (environ && (s == environ[0])) {
  • s += strlen(s);
  • for (i = 1; environ[i]; i++) {
  •  if (environ[i] == s + 1) {
    
  •  s++;
    
  •  s += strlen(s);	/* this one is ok too */
    
  •  }
    
  • }
  • ruby_setenv("", NULL); /* duplicate environ vars */
  • }
    +#endif
  • return s - argv[0];
    +}

static void
set_arg0(val, id)
VALUE val;
@@ -1047,8 +1080,9 @@ set_arg0(val, id)
VALUE progname;
char *s;
long i;

  • int j;
    #if !defined(PSTAT_SETCMD) && !defined(HAVE_SETPROCTITLE)
  • static int len;
  • static int len = 0;
    #endif

    if (origargv == 0) rb_raise(rb_eRuntimeError, "$0 not initialized");
    @@ -1075,27 +1109,7 @@ set_arg0(val, id)
    progname = rb_tainted_str_new(s, i);
    #else
    if (len == 0) {

  • char *s = origargv[0];
  • int i;
  • s += strlen(s);
  • /* See if all the arguments are contiguous in memory */
  • for (i = 1; i < origargc; i++) {
  •  if (origargv[i] == s + 1) {
    
  •  s++;
    
  •  s += strlen(s);	/* this one is ok too */
    
  •  }
    
  •  else {
    
  •  break;
    
  •  }
    
  • }
    -#if defined(USE_ENVSPACE_FOR_ARG0)
  • if (s + 1 == envspace.begin) {
  •  s = envspace.end;
    
  •  ruby_setenv("", NULL); /* duplicate environ vars */
    
  • }
    -#endif
  • len = s - origargv[0];
  • len = get_arglen(origargc, origargv);
    }

    if (i >= len) {
    @@ -1105,8 +1119,10 @@ set_arg0(val, id)
    s = origargv[0] + i;
    *s = '\0';
    if (++i < len) memset(s + 1, ' ', len - i);

  • for (i = 1; i < origargc; i++)
  • origargv[i] = s;
  • for (i = len, j = origargc-1; j > 0 && i > 0; --i, --j) {
  • origargv[j] = origargv[0] + i - 1;
  • *origargv[j] = '\0';
  • }
    progname = rb_tainted_str_new2(origargv[0]);
    #endif
    rb_progname = rb_obj_freeze(progname);

--
|ZnZ(ゼット エヌ ゼット)
|西山和広(Kazuhiro NISHIYAMA)

=end

Actions #2

Updated by Anonymous over 16 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

=begin
Applied in changeset r17849.
=end

Actions

Also available in: Atom PDF

Like0
Like0Like0