summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorPaul Elder <paul.elder@ideasonboard.com>2020-08-26 17:26:47 +0900
committerPaul Elder <paul.elder@ideasonboard.com>2020-10-07 19:17:31 +0900
commit1469d5e26e27bf5fe4d065a6f3c87edd804f2888 (patch)
tree2ffd4611a917ef4f6a375609e607e749d04d65b2 /src
parent7e59bccb35015dea1a3ce6c50efb0cc23157060f (diff)
libcamera: ProcessManager: make ProcessManager lifetime explicitly managed
If any Process instances are destroyed after the ProcessManager is destroyed, then a segfault will occur. Fix this by making the lifetime of the ProcessManager explicit, and make the CameraManager construct and deconstruct (automatically, via a member variable) the ProcessManager. Update the tests accordingly. Signed-off-by: Paul Elder <paul.elder@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se> Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
Diffstat (limited to 'src')
-rw-r--r--src/libcamera/camera_manager.cpp2
-rw-r--r--src/libcamera/process.cpp46
2 files changed, 21 insertions, 27 deletions
diff --git a/src/libcamera/camera_manager.cpp b/src/libcamera/camera_manager.cpp
index 47d56256..b8d3ccc8 100644
--- a/src/libcamera/camera_manager.cpp
+++ b/src/libcamera/camera_manager.cpp
@@ -18,6 +18,7 @@
#include "libcamera/internal/ipa_manager.h"
#include "libcamera/internal/log.h"
#include "libcamera/internal/pipeline_handler.h"
+#include "libcamera/internal/process.h"
#include "libcamera/internal/thread.h"
#include "libcamera/internal/utils.h"
@@ -67,6 +68,7 @@ private:
std::unique_ptr<DeviceEnumerator> enumerator_;
IPAManager ipaManager_;
+ ProcessManager processManager_;
};
CameraManager::Private::Private(CameraManager *cm)
diff --git a/src/libcamera/process.cpp b/src/libcamera/process.cpp
index 994190dc..72b5afe2 100644
--- a/src/libcamera/process.cpp
+++ b/src/libcamera/process.cpp
@@ -41,28 +41,6 @@ LOG_DEFINE_CATEGORY(Process)
* The ProcessManager singleton keeps track of all created Process instances,
* and manages the signal handling involved in terminating processes.
*/
-class ProcessManager
-{
-public:
- void registerProcess(Process *proc);
-
- static ProcessManager *instance();
-
- int writePipe() const;
-
- const struct sigaction &oldsa() const;
-
-private:
- void sighandler(EventNotifier *notifier);
- ProcessManager();
- ~ProcessManager();
-
- std::list<Process *> processes_;
-
- struct sigaction oldsa_;
- EventNotifier *sigEvent_;
- int pipe_[2];
-};
namespace {
@@ -127,8 +105,20 @@ void ProcessManager::registerProcess(Process *proc)
processes_.push_back(proc);
}
+ProcessManager *ProcessManager::self_ = nullptr;
+
+/**
+ * \brief Construct a ProcessManager instance
+ *
+ * The ProcessManager class is meant to only be instantiated once, by the
+ * CameraManager.
+ */
ProcessManager::ProcessManager()
{
+ if (self_)
+ LOG(Process, Fatal)
+ << "Multiple ProcessManager objects are not allowed";
+
sigaction(SIGCHLD, NULL, &oldsa_);
struct sigaction sa;
@@ -145,6 +135,8 @@ ProcessManager::ProcessManager()
<< "Failed to initialize pipe for signal handling";
sigEvent_ = new EventNotifier(pipe_[0], EventNotifier::Read);
sigEvent_->activated.connect(this, &ProcessManager::sighandler);
+
+ self_ = this;
}
ProcessManager::~ProcessManager()
@@ -153,21 +145,21 @@ ProcessManager::~ProcessManager()
delete sigEvent_;
close(pipe_[0]);
close(pipe_[1]);
+
+ self_ = nullptr;
}
/**
* \brief Retrieve the Process manager instance
*
- * The ProcessManager is a singleton and can't be constructed manually. This
- * method shall instead be used to retrieve the single global instance of the
- * manager.
+ * The ProcessManager is constructed by the CameraManager. This function shall
+ * be used to retrieve the single instance of the manager.
*
* \return The Process manager instance
*/
ProcessManager *ProcessManager::instance()
{
- static ProcessManager processManager;
- return &processManager;
+ return self_;
}
/**