Detect and handle exceptions creating capture engine

Currently exceptions from a plugin are not handled when creating
a capture engine.
Catch the exception and try to use another plugin instead of
bailing out.
This was tested with an experimental GStreamer plugin.

Signed-off-by: Frediano Ziglio <fziglio@redhat.com>
Acked-by: Jonathon Jongsma <jjongsma@redhat.com>

1 files changed, 8 insertions, 1 deletions

diff --git a/src/concrete-agent.cpp b/src/concrete-agent.cpp
index 58ce9c4..ca666d7 100644
--- a/src/concrete-agent.cpp
+++ b/src/concrete-agent.cpp
@@ -129,7 +129,14 @@ FrameCapture *ConcreteAgent::GetBestFrameCapture(const std::set<SpiceVideoCodecT
         // check client supports the codec
         if (codecs.find(plugin.second->VideoCodecType()) == codecs.end())
             continue;
-        FrameCapture *capture = plugin.second->CreateCapture();
+
+        FrameCapture *capture;
+        try {
+            capture = plugin.second->CreateCapture();
+        } catch (const std::exception &err) {
+            syslog(LOG_ERR, "Error creating capture engine: %s", err.what());
+            continue;
+        }
         if (capture) {
             return capture;
         }