change vulns.data to vulns.info, fixes #1660

git-svn-id: file:///home/svn/framework3/trunk@9265 4d416f70-5f16-0410-b530-b9f4589650da
unstable
James Lee 2010-05-09 23:32:43 +00:00
parent 8f161576b5
commit abbd6aeef7
2 changed files with 23 additions and 4 deletions

View File

@ -0,0 +1,18 @@
class StandardizeInfoAndData < ActiveRecord::Migration
def self.up
# Remove the host requirement. We'll add the column back in below.
remove_column :vulns, :data
change_table :vulns do |t|
t.string :info, :limit => 65536
end
end
def self.down
remove_column :vulns, :info
change_table :notes do |t|
t.string :data, :limit => 65536
end
end
end

View File

@ -622,14 +622,15 @@ class DBManager
# :name -- the scanner-specific id of the vuln (e.g. NEXPOSE-cifs-acct-password-never-expires) # :name -- the scanner-specific id of the vuln (e.g. NEXPOSE-cifs-acct-password-never-expires)
# #
# opts can contain # opts can contain
# :data -- a human readable description of the vuln, free-form text # :info -- a human readable description of the vuln, free-form text
# :refs -- an array of Ref objects or string names of references # :refs -- an array of Ref objects or string names of references
# #
def report_vuln(opts) def report_vuln(opts)
return if not active return if not active
raise ArgumentError.new("Missing required option :host") if opts[:host].nil? raise ArgumentError.new("Missing required option :host") if opts[:host].nil?
raise ArgumentError.new("Deprecated data column for vuln, use .info instead") if opts[:data]
name = opts[:name] || return name = opts[:name] || return
data = opts[:data] info = opts[:info]
wait = opts.delete(:wait) wait = opts.delete(:wait)
wspace = opts.delete(:workspace) || workspace wspace = opts.delete(:workspace) || workspace
rids = nil rids = nil
@ -662,8 +663,8 @@ class DBManager
host = get_host(:workspace => wspace, :address => addr) host = get_host(:workspace => wspace, :address => addr)
end end
if data if info
vuln = host.vulns.find_or_initialize_by_name_and_data(name, data, :include => :refs) vuln = host.vulns.find_or_initialize_by_name_and_info(name, info, :include => :refs)
else else
vuln = host.vulns.find_or_initialize_by_name(name, :include => :refs) vuln = host.vulns.find_or_initialize_by_name(name, :include => :refs)
end end