fix(ui): handle home dir lookup error in download path fallback

commit: b24c68bb40bbca79b81f00d3cf1c05830032b55b
author: bnema · 9mo ago · 2026-01-08 17:41 UTC
parent: bef9811
signature: GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email

1 file · +8 -3

M internal/ui/app.go +8 -3


@@ -476,9 +476,14 @@ func (a *App) initDownloadHandler(ctx context.Context) {
476	476		}
477	477		}
478	478		if downloadPath == "" {
479		-	// Final fallback to ~/Downloads.
480		-	home, _ := os.UserHomeDir()
481		-	downloadPath = filepath.Join(home, "Downloads")
	479	+	// Final fallback to ~/Downloads, with /tmp as last resort.
	480	+	home, err := os.UserHomeDir()
	481	+	if err != nil {
	482	+	log.Warn().Err(err).Msg("failed to get home dir, using /tmp for downloads")
	483	+	downloadPath = "/tmp"
	484	+	} else {
	485	+	downloadPath = filepath.Join(home, "Downloads")
	486	+	}
482	487		}
483	488
484	489		// Create download event adapter to show toasts.

 		}
 	}
 	if downloadPath == "" {
-		// Final fallback to ~/Downloads.
+		// Final fallback to ~/Downloads, with /tmp as last resort.
-		home, _ := os.UserHomeDir()
+		home, err := os.UserHomeDir()
-		downloadPath = filepath.Join(home, "Downloads")
+		if err != nil {
+			log.Warn().Err(err).Msg("failed to get home dir, using /tmp for downloads")
+			downloadPath = "/tmp"
+		} else {
+			downloadPath = filepath.Join(home, "Downloads")
+		}
 	}
 	// Create download event adapter to show toasts.