From 94c201f265c758fad38eb69c0a5878970119197a Mon Sep 17 00:00:00 2001
From: Andy Staudacher <andy.st@gmail.com>
Date: Tue, 1 Sep 2009 01:17:39 -0700
Subject: XSS escape in form helper and forge where missing.

---
 system/helpers/form.php | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

(limited to 'system/helpers')

diff --git a/system/helpers/form.php b/system/helpers/form.php
index ce8767c5..815eef84 100644
--- a/system/helpers/form.php
+++ b/system/helpers/form.php
@@ -283,15 +283,21 @@ class form_Core {
 					// Inner key should always be a string
 					$inner_key = (string) $inner_key;
 
-					$sel = in_array($inner_key, $selected) ? ' selected="selected"' : '';
-					$input .= '<option value="'.$inner_key.'"'.$sel.'>'.$inner_val.'</option>'."\n";
+                                        $attr = array('value' => $inner_key);
+                                        if (in_array($inner_key, $selected)) {
+                                          $attr['selected'] = 'selected';
+                                        }
+					$input .= '<option '.html::attributes($attr).'>'.html::purify($inner_val).'</option>'."\n";
 				}
 				$input .= '</optgroup>'."\n";
 			}
 			else
 			{
-				$sel = in_array($key, $selected) ? ' selected="selected"' : '';
-				$input .= '<option value="'.$key.'"'.$sel.'>'.$val.'</option>'."\n";
+					$attr = array('value' => $key);
+					if (in_array($key, $selected)) {
+						$attr['selected'] = 'selected';
+					}
+					$input .= '<option '.html::attributes($attr).'>'.html::purify($val).'</option>'."\n";
 			}
 		}
 		$input .= '</select>';
@@ -410,8 +416,9 @@ class form_Core {
 		{
 			$value = arr::remove('value', $data);
 		}
+                // $value must be ::purify
 
-		return '<button'.form::attributes($data, 'button').' '.$extra.'>'.$value.'</button>';
+		return '<button'.form::attributes($data, 'button').' '.$extra.'>'.html::purify($value).'</button>';
 	}
 
 	/**
@@ -455,7 +462,7 @@ class form_Core {
 			$text = ucwords(inflector::humanize($data['for']));
 		}
 
-		return '<label'.form::attributes($data).' '.$extra.'>'.$text.'</label>';
+		return '<label'.form::attributes($data).' '.$extra.'>'.html::purify($text).'</label>';
 	}
 
 	/**
-- 
cgit v1.2.3