Bug#324462: cuetools: missing braces cause command line parsing to always fail

[Lars Wirzenius]

Package: cuetools
Version: 1.3-1
Tags: patch

Attached is a trivial patch to at least allow the -i and -o options to
work. Braces are missing around the final else section when checking the
option arguments.


--- src/tools/cueconvert.c.old  2005-08-22 11:52:44.000000000 +0300
+++ src/tools/cueconvert.c      2005-08-22 11:52:49.000000000 +0300
@@ -87,18 +87,20 @@
                                iformat = CUE;
                        else if (0 == strcmp("toc", optarg))
                                iformat = TOC;
-                       else
+                       else {
                                fprintf(stderr, "%s: illegal format `%s'\n", 
progname, optarg);
                                usage(1);
+                        }
                        break;
                case 'o':
                        if (0 == strcmp("cue", optarg))
                                oformat = CUE;
                        else if (0 == strcmp("toc", optarg))
                                oformat = TOC;
-                       else
+                       else {
                                fprintf(stderr, "%s: illegal format `%s'\n", 
progname, optarg);
                                usage(1);
+                        }
                        break;
                default:
                        usage(1);