CVE-2024-22197 (Go, published 2024-01-11T18:15Z)
Description: value: Nginx-ui is online statistics for Server Indicators?? Monitor CPU usage, memory usage, load average, and disk usage in real-time. The `Home > Preference` page exposes a small list of nginx settings such as `Nginx Access Log Path` and `Nginx Error Log Path`. However, the API also exposes `test_config_cmd`, `reload_cmd` and `restart_cmd`. While the UI doesn't allow users to modify any of these settings, it is possible to do so by sending a request to the API. This issue may lead to authenticated Remote Code Execution, Privilege Escalation, and Information Disclosure. This issue has been patched in version 2.0.0.beta.9.
Severity: nan (CVSS v3 base score: 8.8)
Weakness: CWE-77 \u2014 Improper Neutralization of Special Elements used in a Command ('Command Injection')
The product constructs all or part of a command using externally-influenced input from an upstream component, but it does not neutralize or incorrectly neutralizes special elements that could modify the intended command when it is sent to a downstream component.
Repository: https://github.com/0xJacky/nginx-ui
Files changed:
api/system/settings.go
settings/nginx.go
settings/server.go
Commit message: fix: add protected fields to settings to mitigate high-severity vulnerability

Credits to @jorgectf for the advisories.


Vulnerable code:
    "github.com/0xJacky/Nginx-UI/api"
    "github.com/0xJacky/Nginx-UI/settings"
    "github.com/gin-gonic/gin"
    "net/http"
    c.JSON(http.StatusOK, gin.H{
        "server": settings.ServerSettings,
        "nginx":  settings.NginxSettings,
        "openai": settings.OpenAISettings,
    })
    var json struct {
        Server settings.Server `json:"server"`
        Nginx  settings.Nginx  `json:"nginx"`
        Openai settings.OpenAI `json:"openai"`
    }

    if !api.BindAndValid(c, &json) {
        return
    }

    settings.ServerSettings = json.Server
    settings.NginxSettings = json.Nginx
    settings.OpenAISettings = json.Openai

    settings.ReflectFrom()

    err := settings.Save()
    if err != nil {
        api.ErrHandler(c, err)
        return
    }
    GetSettings(c)
	ConfigDir     string `json:"config_dir"`
	PIDPath       string `json:"pid_path"`
	TestConfigCmd string `json:"test_config_cmd"`
	ReloadCmd     string `json:"reload_cmd"`
	RestartCmd    string `json:"restart_cmd"`
	HttpHost          string `json:"http_host"`
	HttpPort          string `json:"http_port"`
	RunMode           string `json:"run_mode"`
	JwtSecret         string `json:"jwt_secret"`
	NodeSecret        string `json:"node_secret"`
	Email             string `json:"email"`
	Database          string `json:"database"`
	StartCmd          string `json:"start_cmd"`
	Demo              bool   `json:"demo"`
	PageSize          int    `json:"page_size"`

Fixed code:
	"github.com/0xJacky/Nginx-UI/api"
	"github.com/0xJacky/Nginx-UI/settings"
	"github.com/gin-gonic/gin"
	"net/http"
	"reflect"
	c.JSON(http.StatusOK, gin.H{
		"server": settings.ServerSettings,
		"nginx":  settings.NginxSettings,
		"openai": settings.OpenAISettings,
	})
	var json struct {
		Server settings.Server `json:"server"`
		Nginx  settings.Nginx  `json:"nginx"`
		Openai settings.OpenAI `json:"openai"`
	}

	if !api.BindAndValid(c, &json) {
		return
	}

	// todo: omit protected fields when binding
	fillSettings(&settings.ServerSettings, &json.Server)
	fillSettings(&settings.NginxSettings, &json.Nginx)
	fillSettings(&settings.OpenAISettings, &json.Openai)

	settings.ReflectFrom()

	err := settings.Save()
	if err != nil {
		api.ErrHandler(c, err)
		return
	}

	GetSettings(c)
}
func fillSettings(targetSettings interface{}, newSettings interface{}) {
	s := reflect.TypeOf(targetSettings).Elem()
	vt := reflect.ValueOf(targetSettings).Elem()
	vn := reflect.ValueOf(newSettings).Elem()

	// copy the values from new to target settings if it is not protected
	for i := 0; i < s.NumField(); i++ {
		if s.Field(i).Tag.Get("protected") != "true" {
			vt.Field(i).Set(vn.Field(i))
		}
	}
	ConfigDir     string `json:"config_dir" protected:"true"`
	PIDPath       string `json:"pid_path" protected:"true"`
	TestConfigCmd string `json:"test_config_cmd" protected:"true"`
	ReloadCmd     string `json:"reload_cmd" protected:"true"`
	RestartCmd    string `json:"restart_cmd" protected:"true"`
	HttpHost          string `json:"http_host" protected:"true"`
	HttpPort          string `json:"http_port" protected:"true"`
	RunMode           string `json:"run_mode" protected:"true"`
	JwtSecret         string `json:"jwt_secret" protected:"true"`
	NodeSecret        string `json:"node_secret" protected:"true"`
	Email             string `json:"email" protected:"true"`
	Database          string `json:"database" protected:"true"`
	StartCmd          string `json:"start_cmd" protected:"true"`
	Demo              bool   `json:"demo" protected:"true"`
	PageSize          int    `json:"page_size" protected:"true"`