From 102e004727c29072e43dd2c85db7716bdb8d6878 Mon Sep 17 00:00:00 2001
From: Aleksandr Nogikh <nogikh@google.com>
Date: Mon, 13 Jan 2025 19:37:36 +0100
Subject: all: support empty HTTP config

We don't really need an HTTP server when running syz-manager during kernel
image testing and when running syz-diff automatically.

Don't require the config to be set and don't start the HTTP server in
this case.
---
 pkg/manager/http.go   | 3 +++
 pkg/mgrconfig/load.go | 1 -
 2 files changed, 3 insertions(+), 1 deletion(-)

(limited to 'pkg')

diff --git a/pkg/manager/http.go b/pkg/manager/http.go
index 6d4e2ac40..65f0a5714 100644
--- a/pkg/manager/http.go
+++ b/pkg/manager/http.go
@@ -68,6 +68,9 @@ type HTTPServer struct {
 }
 
 func (serv *HTTPServer) Serve() {
+	if serv.Cfg.HTTP == "" {
+		log.Fatalf("starting a disabled HTTP server")
+	}
 	if serv.Pool != nil {
 		serv.Pools = map[string]*vm.Dispatcher{"": serv.Pool}
 	}
diff --git a/pkg/mgrconfig/load.go b/pkg/mgrconfig/load.go
index 602cf4ec8..0624dae00 100644
--- a/pkg/mgrconfig/load.go
+++ b/pkg/mgrconfig/load.go
@@ -145,7 +145,6 @@ func Complete(cfg *Config) error {
 		cfg.TargetArch, "target",
 		cfg.Workdir, "workdir",
 		cfg.Syzkaller, "syzkaller",
-		cfg.HTTP, "http",
 		cfg.Type, "type",
 		cfg.SSHUser, "ssh_user",
 	); err != nil {
-- 
cgit mrf-deployment