「WordPress 4.7.0および4.7.1 に認証なしでコンテンツをアップロードできる深刻な脆弱性」記事へのコメント

PHPの型変換により特定の処理が動いてしまう

[Anonymous Coward]

＞$id (引用者注:中身は文字列'1A') を整数にキャストしているため、get_post(1)が呼ばれ、ID=1 のコンテンツが変更されることになります
いやいやいや
どういう頭の構造してたら(int)"1A"が1になる言語設計しようと思うんだよ…

Re:PHPの型変換により特定の処理が動いてしまう

[Anonymous Coward]

どういう頭の構造してたら(int)"1A"が1になる言語設計しようと思うんだよ…

$ ruby -v
ruby 2.4.0p0 (2016-12-24 revision 57164) [x86_64-linux]
$ ruby -e 'p "1A".to_i'
1

$ perl -v
 
This is perl 5, version 24, subversion 1 (v5.24.1) built for x86_64-linux
(with 30 registered patches, see perl -V for more detail)
 
Copyright 1987-2016, Larry Wall
 
Perl may be copied only under the terms of either the Artistic License or the
GNU General Public License, which may be found in the Perl 5 source kit.
 
Complete documentation for Perl, including FAQ lists, should be found on
this system using "man perl" or "perldoc perl".  If you have access to the
Internet, point your browser at http://www.perl.org/, the Perl Home Page.
 
$ perl -e 'print int("1A");'
1

$ python -V
Python 3.5.2
$ python -c 'print(int("1A"))'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ValueError: invalid literal for int() with base 10: '1A'
 
$ python -c 'print( int("1A",16) )'
26

$ lua -v
Lua 5.1.5  Copyright (C) 1994-2012 Lua.org, PUC-Rio
$ lua -e 'print( tonumber("1A") )'
nil
$ lua -e 'print( tonumber("1A",16) )'
26

御三家で言うと、Pythonだけがまともな頭の構造してるということですね。わかります。

Re:

[Anonymous Coward]

atoi("1A")はいくらになるの！っと

Re:

[Anonymous Coward]

#!ruby

p "1A".to_i

Re:

[Anonymous Coward]

#!perl

print int("1A");

Re:

[Anonymous Coward]

#!ruby
p Integer("1A") #~> ArgumentError: invalid value for Integer(): "1A"
p Integer("1A", 16) #=> 26

Re:

[Anonymous Coward]

> どういう頭の構造してたら
先頭から見ていって、数字と解釈できなくなるところまで変換。
というところじゃなかろうか。

Re:

[Anonymous Coward]

ヘッダが"1"でトレーラが"A"という構造か

Re:

[Anonymous Coward]

$ cat src.cc
#include <iostream>
#include <string>
#include <cstdlib>
#include "boost/lexical_cast.hpp"
 
int main() {
    std::string str = "1A";
    std::cout << "atoi() -> " << std::atoi( str.c_str() ) << std::endl;
    std::cout << boost::lexical_cast<int>(str) << std::endl;
}
 
$ g++ src.cc ; ./a.out
atoi() -> 1
terminate called after throwing an instance of 'boost::exception_detail::clone_impl<boost::exceptio

Re:

[Anonymous Coward]

歴史的な経緯があるんですかね。

ちなみにPASCAL（といってもボーランド系からの流れ）では、例外を投げるStrToIntと、戻り値不定だがエラーがあったことが分かるValがあるようです。

#この15年くらい触ってなかったのでググった。

Re:

[Anonymous Coward]

PHPのキャストはどうなってるんだって読んで行ったら、C言語のstrtod関数も結構大胆なんだね。doubleに変換できない文字があったことはわかるけど、変換自体が失敗するわけじゃなくて、無視するだけなんだ…未知の単位がくっついていて、いちいち引き剥がさなくてもいいようにっていう親切心なのかな？？
http://www9.plala.or.jp/sgwr-t/lib/strtod.html [plala.or.jp]

Re:

[Anonymous Coward]

そのリンク貼ってその感想に至る理由がわからない

Re:

[Anonymous Coward]

perl

print "1A" + "1A"; #2
print "1A" * 1; #1

my $str = "1A";
print ++$str; #2

どこからどうみても1Aは1だよね。

Re:

[Anonymous Coward]

それも酷いが、urlの引数のidの書式のチェックをする気がない？
断じてis_numericは使わない？
モコウ…これでチェックメイト？

Re:

[Anonymous Coward]

あれ？って思ったけどPHP7からはis_numericで16進はエラーになったのか
でもほとんどのところはまだ7じゃないんじゃない？
1Aは数値としては妥当だから7以前だと通しちゃうんじゃないかな

Re:

[Anonymous Coward]

ああ、0xついてないとfalseになるね、失礼しました