[RFC PATCH v5 4/9] libcamera: process: Return error if already running
Barnabás Pőcze
barnabas.pocze at ideasonboard.com
Thu Apr 24 13:40:58 CEST 2025
Returning 0 when a running process is already managed can be confusing
since the parameters might be completely different, causing the caller
to mistakenly assume that the program it specified has been started.
Signed-off-by: Barnabás Pőcze <barnabas.pocze at ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
---
src/libcamera/process.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/libcamera/process.cpp b/src/libcamera/process.cpp
index c22c9af5d..ca9733d77 100644
--- a/src/libcamera/process.cpp
+++ b/src/libcamera/process.cpp
@@ -241,7 +241,7 @@ int Process::start(const std::string &path,
int ret;
if (pid_ > 0)
- return 0;
+ return -EBUSY;
int childPid = fork();
if (childPid == -1) {
--
2.49.0
More information about the libcamera-devel
mailing list