From 97f1f8d922c7b3cb1e849afc29891116a70732c7 Mon Sep 17 00:00:00 2001
From: Kees Monshouwer <mind04@monshouwer.org>
Date: Sun, 31 Aug 2014 21:04:03 +0200
Subject: [PATCH] fix possible fd leak in signpipe worker

---
 pdns/signingpipe.cc | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/pdns/signingpipe.cc b/pdns/signingpipe.cc
index bb164e3405..0756149f38 100644
--- a/pdns/signingpipe.cc
+++ b/pdns/signingpipe.cc
@@ -62,11 +62,8 @@ try
   shs.d_csp->worker(shs.d_id, shs.d_fd);
   return 0;
 }
-catch(PDNSException& pe) {
-  L<<Logger::Error<<"Signing thread died with error "<<pe.reason<<endl;
-}
-catch(std::exception& e) {
-  L<<Logger::Error<<"Signing thread died with error "<<e.what()<<endl;
+catch(...) {
+  L<<Logger::Error<<"unknown exception in signing thread occurred"<<endl;
   return 0;
 }
 
@@ -303,6 +300,11 @@ try
   }
   close(fd);
 }
+catch(PDNSException& pe)
+{
+  L<<Logger::Error<<"Signing thread died because of PDNSException: "<<pe.reason<<endl;
+  close(fd);
+}
 catch(std::exception& e)
 {
   L<<Logger::Error<<"Signing thread died because of std::exception: "<<e.what()<<endl;
-- 
2.47.2