buffer: never unlink controlling socket in cleanup

The pager already unlinks it, so ideally this should change nothing.

(This was causing buffer processes to core dump on OpenBSD, because I
forgot that I removed unlink promise. Better keep code paths the same
when possible...)

1 files changed, 2 insertions, 5 deletions

diff --git a/src/server/buffer.nim b/src/server/buffer.nim
index 178dd94b..cffe33b1 100644
--- a/src/server/buffer.nim
+++ b/src/server/buffer.nim
@@ -1955,11 +1955,8 @@ proc runBuffer(buffer: Buffer) =
 proc cleanup(buffer: Buffer) =
   buffer.pstream.sclose()
   urandom.sclose()
-  # no unlink access on Linux
-  when defined(linux):
-    buffer.ssock.close(unlink = false)
-  else:
-    buffer.ssock.close()
+  # no unlink access on Linux, so just hope that the pager could clean it up
+  buffer.ssock.close(unlink = false)
 
 proc launchBuffer*(config: BufferConfig; url: URL; attrs: WindowAttributes;
     ishtml: bool; charsetStack: seq[Charset]; loader: FileLoader;