[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 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. ;-)

Sieht aber recht sinnstiftend aus.

Acked-by: Max Reitz <max@xxxxxxxxxx>