Project

General

Profile

Backport #8440

backport r40900 (fix invalid read by index underflow.)

Added by nagachika (Tomoyuki Chikanaga) over 6 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
[ruby-dev:47364]

Description

よく確認していませんがバックポートが必要な気がします。
なお r40900 だけだと conflict してすんなり merge できない模様。

Associated revisions

Revision 90490d67
Added by nagachika (Tomoyuki Chikanaga) over 6 years ago

merge revision(s) 39637,40900: [Backport #8440]

    load.c: reorder conditions

    * load.c (loaded_feature_path): reorder conditions so simple

comparision comes first.
* load.c (loaded_feature_path): fix invalid read by index underflow.
the beginning of name is also a boundary as well as just after '/'.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_0_0@41065 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 41065
Added by nagachika (Tomoyuki Chikanaga) over 6 years ago

merge revision(s) 39637,40900: [Backport #8440]

load.c: reorder conditions

* load.c (loaded_feature_path): reorder conditions so simple

comparision comes first.
* load.c (loaded_feature_path): fix invalid read by index underflow.
the beginning of name is also a boundary as well as just after '/'.

History

Updated by nagachika (Tomoyuki Chikanaga) over 6 years ago

r39637 も一緒にするとうまくマージできました。
条件分岐の順番の入れ替えによるちょっとした最適化なので一緒に入れます。

#2

Updated by nagachika (Tomoyuki Chikanaga) over 6 years ago

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

This issue was solved with changeset r41065.
Tomoyuki, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


merge revision(s) 39637,40900: [Backport #8440]

load.c: reorder conditions

* load.c (loaded_feature_path): reorder conditions so simple

comparision comes first.
* load.c (loaded_feature_path): fix invalid read by index underflow.
the beginning of name is also a boundary as well as just after '/'.

Also available in: Atom PDF