アシアルの中の人が技術と思いのたけをつづるブログ

Smartyのバグ ～default_modifiers～

タグ [　 日常　 Smarty　 default_modifiers　 イケメン　 ]
わかりやすいネタのほうがソーシャルブックマーク数が稼げるからこういうネタを掲載するわけじゃないです。
そう、ぼくはただこの土日を使って社内のイケメンリストを作りたかったんです。
硬派なぼくはerror_reportingは常時E_ALLだし、クロスサイト・スクリプティング（以下XSS）の脆弱性を産むようなミスは犯したりしないゼ。
へなちょこでもいい。たくましいプログラミングをしたいんだ。

しかし悲劇は起こりました。

１　イケメン一覧スクリプトを作成する

まず、Asial社内のイケメンリストを配列に持ち、Smartyのassignメソッドで配列をテンプレート変数に割り当て。
また、XSSの脆弱性を未然に防ぐため、Smartyのdefault_modifiersにescapeをセットし、自動的にhtmlspecialchars関数がかかるように設定します。

ikemen_list.php
  1. <?php
  2.  
  3. // イケメンリストの配列を作成
  4. $ikemen_list[0]["name"]       = "海原";
  5. $ikemen_list[0]["nickname"]   = "saity";
  6. $ikemen_list[0]["honey_flg"]  = "0";
  7. $ikemen_list[1]["name"]       = "田中";
  8. $ikemen_list[1]["nickname"]   = "massie";
  9. $ikemen_list[1]["honey_flg"]  = "1";
  10. $ikemen_list[2]["name"]       = "森川";
  11. $ikemen_list[2]["nickname"]   = "joe";
  12. $ikemen_list[2]["honey_flg"]  = "1";
  13. $ikemen_list[3]["name"]       = "松田";
  14. $ikemen_list[3]["nickname"]   = "mattsun";
  15. $ikemen_list[3]["honey_flg"]  = "0";
  16.  
  17. require_once("../libs/Smarty.class.php");
  18. $smarty = new Smarty();
  19. $smarty->template_dir = "/home/saito/templates";
  20. $smarty->compile_dir  = "/home/saito/templates_c";
  21. $smarty->default_modifiers = array('escape');
  22.  
  23. $smarty->assign("ikemen_list",$ikemen_list);
  24. $smarty->display("ikemen_list.html");



次に、テンプレート側のスクリプトを組みました。
ちゃんと値がassignされたかを確認するために、{debug}関数を挟みます。

ikemen_list.html
  1. <html>
  2. <head>
  3. <title>社内モテリスト</title>
  4. <body>
  5. {debug}
  6.  
  7. <h1>アシアルのイケメン一覧表</h1>
  8. {foreach from=$ikemen_list item=ikemen name="motefoobar"}
  9.   {$smarty.foreach.motefoobar.iteration}:
  10.   {$ikemen.name}:
  11.   {$ikemen.nickname}:
  12.   {if isset($ikemen.honey_flg) && $ikemen.honey_flg}彼女有{else}彼女無{/if}
  13.   <br>
  14. {/foreach}

そして実行。


２　悲劇の始まり ～Smartyのバグ～


ナニィィィ！！？syntax error（構文エラー）だとォ？
どこを何度見ても構文を間違えている様子はないですよね。

仕方ないので
50^50A^50AAF14B%%ikemen_list.html.php
このコンパイル後の24行目で、何が起こっているのかを確かめてみることにします。
問題となっているのがこのくだり。↓
  1. <?php if (isset ( ((is_array($_tmp=$this->_tpl_vars['ikemen']['honey_flg'])) ? $this->_run_mod_handler('escape', true, $_tmp) : smarty_modifier_escape($_tmp)) | ) && ((is_array($_tmp=$this->_tpl_vars['ikemen']['honey_flg'])) ? $this->_run_mod_handler('escape', true, $_tmp) : smarty_modifier_escape($_tmp))): ?>彼女有<?php else: ?>彼女無<?php endif; ?>

変数の中身を判定して、修飾子の処理を噛まそうとしてます。
このif文が構文エラーとなっているわけですが、要約すると
  1. if (isset((is_array(($a) ? $b : $c)))) {

さらにもうちょっと解りやすく噛み砕くと、
  1. if (isset(($a = $b))){

コレが構文エラーの問題です。
phpマニュアルでisset関数を調べてみると、『Warning: isset() は何らかの値が渡された変数の場合のみ動作します。そうでない場合、パースエラーとなります。』とありました。isset関数で、式を評価してはいけませんね！

対策として、修飾子を実行させないよう
  1. {if isset($ikemen.honey_flg|smarty:nodefaults) && $ikemen.honey_flg}彼女有{else}彼女無{/if}
として回避しました。

そして再実行


３　default_modifiersを設定していると{debug}もダメ


またsyntax error！？
どうせ先ほどと同じ問題だと思ったので、中身を見てみると案の定。
しかし、この問題を回避するのは難しそうです。
default_modifiersをセットすると、{debug}関数が使えないことが判明しました。
解決方法があったら、どなたか教えてください。

気を取り直し、{debug}を取り除いて再実行


４　{foreach}する配列もdefault_modifiersの対象となる


どうやら今度は、escape修飾子関数内で問題が起こっているようでした。
プラグインの中の
modifier.escape.php
を調べてみることにしました。
  1. function smarty_modifier_escape($string, $esc_type = 'html', $char_set = 'ISO-8859-1')
  2. {
  3.     switch ($esc_type) {
  4.         case 'html':
  5.             return htmlspecialchars($string, ENT_QUOTES, $char_set);
  6. 以下略
この第1引数に配列が渡ってきちゃうと、htmlspecialchars関数に配列を渡すことになってしまいます。
詳しくコンパイルファイルを調べてみると、やはりdefault_modifiersが悪さをしていました。
ループしたい変数が配列の場合、_run_mod_handlerというメソッド内で配列の中身をもすべて分解してdefault_modifire処理をかけようとします。
しかし_run_mod_handlerメソッド内では、1次元配列までにしか対応していませんでした。だから、今回のように2次元配列である$ikemen_listをループする場合、1次元配列にescape修飾子処理をかけようとしてWarningが出ちゃうんです。

かなり付け焼刃的な措置ではありますが、配列が渡ってきた場合は無視するよう追記します。
  1. function smarty_modifier_escape($string, $esc_type = 'html', $char_set = 'ISO-8859-1')
  2. {
  3.     // 配列が渡ってきたら何もせずにreturn
  4.     if (is_array($string)) return $string;
  5.     switch ($esc_type) {
  6.         case 'html':
  7.             return htmlspecialchars($string, ENT_QUOTES, $char_set);
  8. 以下略

本当なら、多次元配列に対応できるように修正すべきなんだけど、めんどうなので「あとでやる」。
（ここはむしろ新ジャンル「だれかやって」）

ということで、改めて実行


ぃよっしゃー！


５　まとめ

・$default_modifiersを設定すると、{debug}が使えない
　対策→わからないのでだれか教えてください

・$default_modifiersを設定すると、2次元配列を{foreach}できない
　対策→plugin/modifier.escape.phpを配列が渡ってきても大丈夫なように改造する

・$default_modifiersを設定すると、{if isset($variable)}のように、if文内でisset関数が使えない
　対策→{if isset($variable|smarty:nodefaults)}としてdefault_modifiersを無効にする

コメント

    • たぶん
      $ikemen_list[4]["name"] = "かめもと";
      $ikemen_list[4]["nickname"] = "yudoufu";
      $ikemen_list[4]["honey_flg"] = "0";
      これが足りないせいじゃないかな(｀・ω・´)
    • by かめもと | 2007年12月02日 04:38
    • その後に、
      $ikemen_list[5]["name"] = "笹亀";
      $ikemen_list[5]["nickname"] = "sasa";
      $ikemen_list[5]["honey_flg"] = "1";
      きっとこれも足りないから動かない(；ﾟДﾟ)
    • by 笹亀 | 2007年12月02日 04:46
    • ちっ、motefoobarにはだれも食い付かなかったか。
      もっと大胆に行かなきゃダメか。
    • by saity | 2007年12月11日 03:00
    • トラックバックが入るのは時間差でしょうか。{debug}の件直ったので良かったらこちらをどうぞ。
      http://nonn-et-twk.net/twk/node/133
    • by twk | 2007年12月14日 20:12
    • * @author Monte Ohrt <monte at ohrt dot com>
      ↑この人に、バグ報告のメールしておきましたよん。
    • by 山田 | 2008年04月14日 13:33
    • 山田さん？ご報告ありがとうございます☆
      どちらの山田さんでしょう？僕の生徒さん？本をいっぱい出版されている？？
      気になる気になるとき気になれば気になろう…。
    • by Saity | 2008年04月17日 01:15
    • > ご報告ありがとうございます☆
      いえいえ。でも音沙汰がありません・・・。

      > どちらの山田さんでしょう？
      PHP暦半年のSaityさんの知らない山田です。
      でも僕の先生はSaityさんを知ってるそうです。
      （こういうこと書くと、よけいに気になりますね。）


      ところで、今の案件で、Smartyのdefault_modifiers使ってます。
      痛いバグは、Saityさんのバグ措置をコペりました。
      もう、20機能ほど実装しましたが、問題なく動いています。

      このdefault_modifiers、滅茶苦茶便利じゃないですか！？
      なぜこんな便利な機能があるのに、Smartyチームは
      これを前面に押し出さないんでしょう。
      たった一行の設定で、わずらわしいXSS対策作業が
      ほぼ完了しますよね。
      僕がSmartyチームの一員だったら、名刺に
      「default_modifiersのSmarty！」と印刷しておきたいくらいです。

      最近は、フレームワーク／VIEWはPHPがはやってきていますが、
      PHPのコードをPHPでパースすることは大変なので、このような
      ことってできないですよね？（多分）
    • by 山田 | 2008年05月02日 07:52
    • > このようなことってできないですよね？（多分）
      symfonyならできるんですね。失礼しました。
      CakePHPにh()なんてものがあるので無理かと思ってました。。。
    • by 山田 | 2008年05月11日 00:54

コメントフォーム

認証
captcha_key
 

トラックバック

トラックバックURI :

最近の記事