summaryrefslogtreecommitdiffstats
path: root/app
diff options
context:
space:
mode:
authorJean-Philippe Lang <jp_lang@yahoo.fr>2012-03-06 20:34:38 +0000
committerJean-Philippe Lang <jp_lang@yahoo.fr>2012-03-06 20:34:38 +0000
commitefddab48bed65c55c57ac9b95c40a42f059a136f (patch)
tree947fd0f3daf62c964cb572b42c65e71b6318463c /app
parentfef2e4b67252415fefe54da856db3a77c3143dfe (diff)
downloadredmine-efddab48bed65c55c57ac9b95c40a42f059a136f.tar.gz
redmine-efddab48bed65c55c57ac9b95c40a42f059a136f.zip
Prevent mass-assignment when adding/updating a wiki (#10390).
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@9138 e93f8b46-1217-0410-a6f0-8f06a7374b81
Diffstat (limited to 'app')
-rw-r--r--app/controllers/wikis_controller.rb2
-rw-r--r--app/models/wiki.rb3
2 files changed, 4 insertions, 1 deletions
diff --git a/app/controllers/wikis_controller.rb b/app/controllers/wikis_controller.rb
index 523b80678..251b2a963 100644
--- a/app/controllers/wikis_controller.rb
+++ b/app/controllers/wikis_controller.rb
@@ -22,7 +22,7 @@ class WikisController < ApplicationController
# Create or update a project's wiki
def edit
@wiki = @project.wiki || Wiki.new(:project => @project)
- @wiki.attributes = params[:wiki]
+ @wiki.safe_attributes = params[:wiki]
@wiki.save if request.post?
render(:update) {|page| page.replace_html "tab-content-wiki", :partial => 'projects/settings/wiki'}
end
diff --git a/app/models/wiki.rb b/app/models/wiki.rb
index 1b4301707..6cdfa246d 100644
--- a/app/models/wiki.rb
+++ b/app/models/wiki.rb
@@ -16,6 +16,7 @@
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
class Wiki < ActiveRecord::Base
+ include Redmine::SafeAttributes
belongs_to :project
has_many :pages, :class_name => 'WikiPage', :dependent => :destroy, :order => 'title'
has_many :redirects, :class_name => 'WikiRedirect', :dependent => :delete_all
@@ -25,6 +26,8 @@ class Wiki < ActiveRecord::Base
validates_presence_of :start_page
validates_format_of :start_page, :with => /^[^,\.\/\?\;\|\:]*$/
+ safe_attributes 'start_page'
+
def visible?(user=User.current)
!user.nil? && user.allowed_to?(:view_wiki_pages, project)
end