fix non-constant format string in call to fmt.Errorf

Author: iQQBot

components/local-app/pkg/selfupdate/selfupdate.go

@@ -136,7 +136,7 @@ func DownloadManifest(ctx context.Context, baseURL string) (res *Manifest, err e
 	defer resp.Body.Close()
 
 	if resp.StatusCode != http.StatusOK {
-		return nil, fmt.Errorf(resp.Status)
+		return nil, fmt.Errorf("%s", resp.Status)
 	}
 
 	var mf Manifest

components/supervisor/cmd/init.go

@@ -92,7 +92,7 @@ var initCmd = &cobra.Command{
 				if ignoreUnexpectedExitCode.Load() {
 					return
 				}
-				log.WithError(fmt.Errorf(logs)).Fatal("supervisor run error with unexpected exit code")
+				log.WithError(fmt.Errorf("%s", logs)).Fatal("supervisor run error with unexpected exit code")
 			}
 		}
 		go func() {

components/ws-daemon/cmd/content-initializer/main.go

@@ -26,7 +26,7 @@ func main() {
 	err := content.RunInitializerChild(statsFd)
 	if err != nil {
 		errfd := os.NewFile(content.RUN_INITIALIZER_CHILD_ERROUT_FD, "errout")
-		_, _ = fmt.Fprintf(errfd, err.Error())
+		_, _ = fmt.Fprintf(errfd, "%s", err.Error())
 
 		os.Exit(content.FAIL_CONTENT_INITIALIZER_EXIT_CODE)
 	}

components/ws-daemon/pkg/controller/housekeeping.go

@@ -70,7 +70,7 @@ func (h *Housekeeping) doHousekeeping(ctx context.Context) (errs []error) {
 
 		var err error
 		if len(msgs) > 0 {
-			err = fmt.Errorf(strings.Join(msgs, ". "))
+			err = fmt.Errorf("%s", strings.Join(msgs, ". "))
 		}
 		tracing.FinishSpan(span, &err)
 	}()

components/ws-manager-mk2/service/imagespec.go

@@ -32,7 +32,7 @@ func (is *WorkspaceImageSpecProvider) GetImageSpec(ctx context.Context, req *reg
 		return nil, status.Errorf(codes.NotFound, "not found")
 	}
 	if err != nil {
-		return nil, status.Errorf(codes.Internal, err.Error())
+		return nil, status.Errorf(codes.Internal, "%s", err.Error())
 	}
 
 	return &regapi.GetImageSpecResponse{

components/ws-manager-mk2/service/manager.go

@@ -195,7 +195,7 @@ func (wsm *WorkspaceManagerServer) StartWorkspace(ctx context.Context, req *wsma
 	storage, err := class.Container.Limits.StorageQuantity()
 	if err != nil {
 		msg := fmt.Sprintf("workspace class %s has invalid storage quantity: %v", class.Name, err)
-		return nil, status.Errorf(codes.InvalidArgument, msg)
+		return nil, status.Errorf(codes.InvalidArgument, "%s", msg)
 	}
 
 	annotations := make(map[string]string)
@@ -690,7 +690,7 @@ func (wsm *WorkspaceManagerServer) TakeSnapshot(ctx context.Context, req *wsmana
 		}
 
 		if sso.Status.Error != "" {
-			return true, fmt.Errorf(sso.Status.Error)
+			return true, fmt.Errorf("%s", sso.Status.Error)
 		}
 
 		if sso.Status.URL != "" {

components/ws-proxy/pkg/sshproxy/server.go

@@ -600,7 +600,7 @@ func workspaceSSHUsername(ctx context.Context, workspaceIP string, workspacekitP
 	}
 
 	if resp.StatusCode != http.StatusOK {
-		return "", fmt.Errorf(fmt.Sprintf("unexpected status: %v (%v)", string(result), resp.StatusCode))
+		return "", fmt.Errorf("unexpected status: %v (%v)", string(result), resp.StatusCode)
 	}
 
 	return string(result), nil