diff options
| author | Michal Ratajsky <[email protected]> | 2016-01-04 20:31:14 +0100 |
|---|---|---|
| committer | Michal Ratajsky <[email protected]> | 2016-01-04 20:31:14 +0100 |
| commit | 9772797c31ebed2417b42a9389caae1b16847e86 (patch) | |
| tree | 0257271d6f8387ce89984bf8341b11186381c8d5 /backends/oss/oss-switch.c | |
| parent | d32ca3f420d036cd750fc1aad7f95da40559cc5d (diff) | |
| download | libmatemixer-9772797c31ebed2417b42a9389caae1b16847e86.tar.bz2 libmatemixer-9772797c31ebed2417b42a9389caae1b16847e86.tar.xz | |
Improve error checking in many places
Diffstat (limited to 'backends/oss/oss-switch.c')
| -rw-r--r-- | backends/oss/oss-switch.c | 11 |
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; |