nghttpx: Fix mruby parameter validation

ruanrj · Sep 30, 2018 · fc7489e · fc7489e
1 parent 87ac872
commit fc7489e
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/src/shrpx_config.cc b/src/shrpx_config.cc
@@ -1056,12 +1056,14 @@ int parse_mapping(Config *config, DownstreamAddrConfig &addr,
       // If some backends do not specify mruby file, and there is at
       // least one backend with mruby file, it is used for all
       // backends in the group.
-      if (g.mruby_file.empty()) {
-        g.mruby_file = make_string_ref(downstreamconf.balloc, params.mruby);
-      } else if (g.mruby_file != params.mruby) {
-        LOG(ERROR) << "backend: mruby: multiple different mruby file found in "
-                      "a single group";
-        return -1;
+      if (!params.mruby.empty()) {
+        if (g.mruby_file.empty()) {
+          g.mruby_file = make_string_ref(downstreamconf.balloc, params.mruby);
+        } else if (g.mruby_file != params.mruby) {
+          LOG(ERROR) << "backend: mruby: multiple different mruby file found "
+                        "in a single group";
+          return -1;
+        }
       }
 
       g.addrs.push_back(addr);