Improve error checking in many places

1 files changed, 9 insertions, 2 deletions

diff --git a/backends/oss/oss-switch.c b/backends/oss/oss-switch.c
index 6b78833..ba07d36 100644
--- a/backends/oss/oss-switch.c
+++ b/backends/oss/oss-switch.c
@@ -107,13 +107,20 @@ oss_switch_new (OssStream   *stream,
                 GList       *options)
 {
     OssSwitch *swtch;
+    gint       newfd;
 
     g_return_val_if_fail (OSS_IS_STREAM (stream), NULL);
     g_return_val_if_fail (name != NULL, NULL);
     g_return_val_if_fail (label != NULL, NULL);
-    g_return_val_if_fail (fd != -1, NULL);
     g_return_val_if_fail (options != NULL, NULL);
 
+    newfd = dup (fd);
+    if (newfd == -1) {
+        g_warning ("Failed to duplicate file descriptor: %s",
+                   g_strerror (errno));
+        return NULL;
+    }
+
     swtch = g_object_new (OSS_TYPE_SWITCH,
                           "name", name,
                           "label", label,
@@ -122,7 +129,7 @@ oss_switch_new (OssStream   *stream,
                           NULL);
 
     /* Takes ownership of options */
-    swtch->priv->fd      = dup (fd);
+    swtch->priv->fd      = newfd;
     swtch->priv->options = options;
 
     return swtch;