[Groonga-commit] groonga/groonga-query-log at afb2b90 [master] check-crash: add missing nil check

Back to archive index
Kouhei Sutou null+****@clear*****
Thu Jan 10 09:51:09 JST 2019


Kouhei Sutou	2019-01-10 09:51:09 +0900 (Thu, 10 Jan 2019)

  Revision: afb2b9079228125e54760d933f25afa6efcbeba3
  https://github.com/groonga/groonga-query-log/commit/afb2b9079228125e54760d933f25afa6efcbeba3

  Message:
    check-crash: add missing nil check
    
    statistic.command is nil when request path is "/".

  Modified files:
    lib/groonga-query-log/command/check-crash.rb

  Modified: lib/groonga-query-log/command/check-crash.rb (+6 -3)
===================================================================
--- lib/groonga-query-log/command/check-crash.rb    2018-11-20 11:09:02 +0900 (f35eddc)
+++ lib/groonga-query-log/command/check-crash.rb    2019-01-10 09:51:09 +0900 (7db757a)
@@ -209,7 +209,10 @@ module GroongaQueryLog
         end
 
         def check_query_log_statistic(path, statistic)
-          case statistic.command.command_name
+          command = statistic.command
+          return if command.nil?
+
+          case command.command_name
           when "load"
             @flushed = false
             @unflushed_statistics << statistic
@@ -220,10 +223,10 @@ module GroongaQueryLog
             @flushed = false
             @unflushed_statistics << statistic
           when "io_flush"
-            check_io_flush(statistic.command)
+            check_io_flush(command)
           when "database_unmap"
             @unflushed_statistics.reject! do |statistic|
-              statistic.command.name == "load"
+              command.name == "load"
             end
           when "table_list", "column_list"
             # ignore
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.osdn.me/mailman/archives/groonga-commit/attachments/20190110/d0dfdf5e/attachment-0001.html>


More information about the Groonga-commit mailing list
Back to archive index