OliveTin gives access to predefined shell commands from a web interface. Prior to 3000.11.2, when the saveLogs feature is enabled, OliveTin persists execution log entries to disk. The filename used for these log files is constructed in part from the user-supplied UniqueTrackingId field in the StartAction API request. This value is not validated or sanitized before being used in a file path, allowing an attacker to use directory traversal sequences (e.g., ../../../) to write files to arbitrary locations on the filesystem. This vulnerability is fixed in 3000.11.2.

Affected: Go: github.com/OliveTin/OliveTin

When the `saveLogs` feature is enabled, OliveTin persists execution log entries to disk. The filename used for these log files is constructed in part from the user-supplied `UniqueTrackingId` field in the `StartAction` API request. This value is not validated or sanitized before being used in a file path, allowing an attacker to use directory traversal sequences (e.g., `../../../`) to write files to arbitrary locations on the filesystem. ### Affected Code

**Entry point — `service/internal/api/api.go` (line 130):**

The `UniqueTrackingId` from the API request is passed directly to the executor without validation:

```go execReq := executor.ExecutionRequest{ Binding: pair, TrackingID: req.Msg.UniqueTrackingId, // user-controlled, no validation // ... } ```

**Tracking ID accepted as-is — `service/internal/executor/executor.go` (lines 508–512):**

The tracking ID is only replaced with a UUID if it is empty or a duplicate. Any other string, including one containing path separators, is accepted:

```go _, isDuplicate := e.GetLog(req.TrackingID)

if isDuplicate || req.TrackingID == "" { req.TrackingID = uuid.NewString() } ```

**Filename construction — `service/internal/executor/executor.go` (line 1042):**

The tracking ID is interpolated directly into the log filename:

```go filename := fmt.Sprintf("%v.%v.%v", req.logEntry.ActionTitle, req.logEntry.DatetimeStarted.Unix(), req.logEntry.ExecutionTrackingID, ) ```

**File write — `service/internal/ex…