ruby-on-rails - 如何修复 Ruby 2.0.0p645 的段错误

Question

当我在 Sinatra/Ruby 应用程序中解析联盟提要时，出现错误:

/opt/rh/ruby200/root/usr/share/ruby/net/http/response.rb:368: [BUG]
Segmentation fault ruby 2.0.0p645 (2015-04-13) [x86_64-linux]

我想知道这是 Ruby 的错误还是代码有问题，如果是，我该如何修复它？

Link to the error

这是解析联盟提要的代码:

        feeds.each { |name, hash|
            puts "=== PARSING #{name.upcase} FEED ==="
            start = Time.now
            open(hash[:url]) { |feed|
                send(hash[:action], feed)
            }
            duration = Time.now - start
            puts "Feed syndication completed in #{duration.to_s} seconds."
            puts
        }

        # Close DB connection
        puts "Disconnecting"
        @db.disconnect
    end

    def parseAllianceData(xml)
        start = Time.now
        allianceData = XMLObject.new xml
        duration = Time.now - start

        puts "XML parsed in #{duration.to_s} seconds."
        puts "Alliances found: #{allianceData.alliances.count}"

        @db[:feeds].insert(
            :generated_at => allianceData.server.datagenerationdatetime,
            :type => "Alliance",
            :is_current => true)

        start = Time.now
        allianceData.alliances.each { |alliance|
            capital_last_moved_at = (alliance.alliancecapitallastmoved rescue nil)
            taxrate_last_changed_at = (alliance.alliancetaxratelastchanged rescue nil)
            @db[:alliance].insert(
                :id => alliance.alliance[:id],
                :ticker => alliance.allianceticker,
                :name => alliance.alliance,
                :founded_at => alliance.foundeddatetime,
                :founded_by_player_id => alliance.foundedbyplayerid[:id],
                :capital_town_id => alliance.alliancecapitaltownid[:id],
                :member_count => alliance.membercount,
                :total_population => (alliance.totalpopulation rescue 0),
                :tax_rate => (alliance.alliancetaxrate.to_i) / 100.0,
                :tax_rate_last_changed_at => taxrate_last_changed_at,
                :capital_town_last_moved_at => capital_last_moved_at)

            alliance.roles.each { |role|
                @db[:alliance_roles].insert(
                    :id => role.role[:id],
                    :name => role.role,
                    :alliance_id => alliance.alliance[:id],
                    :hierarchy_id => role.heirarchy[:id])
            }
        }
        duration = Time.now - start
        puts "Database populated in #{duration.to_s} seconds."

Answer 1

我在您的示例中发现了一行危险的代码:

send(hash[:action], feed)

它从外部源 (hash[:action]) 获取一些字符串并将其转换为方法调用。这是非常危险的，因为你永远不知道你会得到什么字符串。那里可能有一个字符串无法进入方法调用，因此 Ruby 崩溃。

我建议明确检查所有支持的操作和调用方法。例如，您可以使用 case 语句来完成。

action = hash[:action]

case action
when 'action1'
  call_method1
when 'action2'
  call_method2
else
  puts "unsupported action: #{action}"
end