]> source.dussan.org Git - gitea.git/commit
cmd/serv: actually exit after fatal errors (#7458)
authorAllen Wild <aswild@users.noreply.github.com>
Sat, 13 Jul 2019 20:38:18 +0000 (16:38 -0400)
committerzeripath <art27@cantab.net>
Sat, 13 Jul 2019 20:38:18 +0000 (21:38 +0100)
commit2186a99c768cf4511d9fbed0781d94d892199873
tree67580be1d061e128f905d47c7d031db39dd956a2
parent50d8d171f01b82f58aa38c16495590c798f3dccb
cmd/serv: actually exit after fatal errors (#7458)

Regression in 356854fc5f8d7d1a7e4d68c9e00929e9ce8aa867, where
a log.Fatal call was removed. log.Fatal calls os.Exit(1) as intended,
but without it the fail() function returns normally and execution
continues past the fatal error, causing a panic.

This is visible as a go panic log and stack trace returned to the SSH
client, which is not only ugly, it leaks server and build system
information.

Fix by removing the stray return statement so that the fail() function
always calls os.Exit(1).

Fixes: https://github.com/go-gitea/gitea/issues/7457
Signed-off-by: Allen Wild <allenwild93@gmail.com>
cmd/serv.go