BugTrack/2285

エラーメッセージ用の定数 xxx_USAGE の中にエスケープされていないものがある†

ページ: BugTrack2
投稿者: 名無しさん
優先順位: 重要
状態: 提案
カテゴリー: 本体バグ
投稿日: 2007-11-10 (土) 13:20:55
バージョン:

メッセージ†

プラグインでエラーの発生時にUSAGE を返すものがありますが、& とエスケープせずに&をそのまま使っている物があります。

define('PLUGIN_COLOR_USAGE', '&color(foreground[,background]){text};');

cvs:plugin/edit.inc.php (1.46)

function plugin_edit_inline()
{
	static $usage = '&edit(pagename#anchor[[,noicon],nolabel])[{label}];';

cvs:plugin/ruby.inc.php (1.6)

define('PLUGIN_RUBY_USAGE', '&ruby(pronunciation){words};');

cvs:plugin/size.inc.php (1.10)

define('PLUGIN_SIZE_USAGE', '&size(px){Text you want to change};');

追伸: edit.inc.php は、ついでにBugTrack2/260の件もお願いします。

チェックありがとうございます。それぞれ修正 + αしておきました。可能な範囲でご確認をお願いします。 -- henoheno 2007-11-11 (日) 22:48:49
- cvs:plugin/color.inc.php (1.23-1.24)
- cvs:plugin/edit.inc.php (1.47)
- cvs:plugin/ruby.inc.php (1.7)
- cvs:plugin/size.inc.php (1.11) -- Percentageにも対応させた
size.inc.php (1.11) にちょっとした不具合があります。正規表現パターンが、
```
define('PLUGIN_SIZE_REGEX', '/^(?:([0-9]+(?:px)?)|([0-9]+)%)$/i');
```
となっていて、キャプチャする内容に'px' を含めているので、
&size(12px){word}; と書いた時の出力が、'<span style="font-size:12pxpx;...（省略）' とpx が2重に出力されてしまいます。 -- 2007-11-12 (月) 20:18:47
確認ありがとうございます。こんなんでいかがでしょうか。別件で、考慮し忘れていたintval()なども追加し、もう少々柔らかくしてみました -- henoheno 2007-11-14 (水) 00:00:19
- cvs:plugin/size.inc.php (1.12)
え～と、またsize プラグインの正規表現に疑問が・・・。
```
define('PLUGIN_SIZE_REGEX',
	'/^(?:' .
	'([0-9]+(?:\.[0-9]+)?)(?: *px)?' . '|' .	// Pixel (default)
	'([0-9.]+(?:\.[0-9]+)?) *%' .				// Percentage
	')$/i');
```
パーセントの部分が「0～9 か.（ピリオド）が1つ以上続いた後に・・・」となっているので、&size(120.45.87..01%){word}; とピリオドをいくつ書いてもOK になっています。
intval() しているので、たいした問題では無いかもしれませんが。 -- 2007-11-14 (水) 23:47:00
たびたびレビューいただきありがとうございます。それはピリオドの入力を認めるコードを遷移させていた時の名残です。(?:\.[0-9]+)? に集中していて左側を見ていませんでした。比較対照しやすい形に直しておいて良かった (^^; -- henoheno 2007-11-15 (木) 23:31:39
- cvs:plugin/size.inc.php (1.13)

（大文字小文字無視の）USAGE でしか検索してなかったので、見逃してた分を挙げます。 -- 2007-11-16 (金) 19:03:53
2007-11-16 (金) 00:00:00 のCVS版

plugin\counter.inc.php(31):		return '&counter([total|today|yesterday]);';

plugin\dump.inc.php(85):		$msg = 'dump & restore';

plugin\new.inc.php(35):		if (func_num_args() > 2) return '&new([nodate]){date};';
plugin\new.inc.php(39):		if ($timestamp === -1) return '&new([nodate]){date}: Invalid date string;';
plugin\new.inc.php(63):				return '&new(pagename/[,nolink]): No such pages;';
plugin\new.inc.php(70):				return '&new(pagename[,nolink]): No such page;';

plugin\newpage.inc.php(54):			'?cmd=read&page=' . $r_page . '&refer=' . $r_refer);

plugin\online.inc.php(58):			$error = '&online: ' . $error . ';';

dump, newpage プラグインは、BugTrack2/260 も・・・（略） -- 2007-11-16 (金) 19:23:52
修正（header() などは生データを扱うので、変換すると正しい場所に飛ばないです・・・） -- 2009-06-16 (火) 23:59:42

追加（USAGE ですらないですが、とりあえずココに） -- 2009-06-16 (火) 23:59:42

plugin\spam.inc.php(31):		'<a href="'. $script . '&mode=pages' . '">Pages</a>' . "\n"
plugin\spam.inc.php(45):	$script  = get_script_uri() . '?plugin=spam&mode=pages';

total:
today:
yesterday:
now: 1

エラーメッセージ用の定数 xxx_USAGE の中にエスケープされていないものがある†

メッセージ†

最新の16件