[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [tyndur-devel] [PATCH 1/2] shell: bincat: Kein fseek auf NULL aufrufen



On Sun, Dec 11, 2011 at 10:11:07PM +0100, Max Reitz wrote:
> On 10.12.2011 14:18, Kevin Wolf wrote:
> >! shell: Wenn bincat die Datei nicht öffnen kann, muss es sofort einen
> >   Fehler zurückgeben und nicht erst noch den NULL-Pointer an fseek
> >   weitergeben, damit es garantiert rot gibt.
> >
> >Signed-off-by: Kevin Wolf<kevin@xxxxxxxxxx>
> [...]
> >
> >+    if (file == NULL) {
> >+        printf(TMS(bincat_opening_error,
> >+            "Konnte '%s' nicht zum lesen öffnen!\n"), path);
> Na, eigentlich sollte das "zum Lesen" sein. ;-)

Ist zwar nur verschobener Code, aber ich fix das dann gleich noch vor
dem git push.