lsyncd.c:966: warning: comparison with string literal results in unspecified behavior

[403]

moins,

an der Stelle steht:

965:#ifdef XML_CONFIG
966-    if (opts->conf_filename && opts->conf_filename != DEFAULT_CONF_FILENAME) {
967-            s_free(opts->conf_filename);
968-    }
969-    opts->conf_filename = DEFAULT_CONF_FILENAME;
970-#endif

Ich weiss, dass die Warnung mit dem CFLAG -Wno-address
verschwinden lassen kann, aber das behebt nur das Symptom.

Wie fixt man das richtig?


Gruss

http://nopaste.info/c1d9c99dfc.html

[sommerfee]

opts->conf_filename != DEFAULT_CONF_FILENAME

durch

strcmp( opts->conf_filename, DEFAULT_CONF_FILENAME ) != 0

ersetzen.

Ganz richtig ist das aber auch nicht, da dann die Zeichenkette nicht freigegeben wird, wenn opts->conf_filename vom User gewollt den gleichen Inhalt hat. Ich finde die Programmierung etwas merkwürdig, man hätte sich keinen Zacken aus der Krone gebrochen, wenn man statt opts->conf_filename = DEFAULT_CONF_FILENAME eine Zuweisung mit Speicheralloziierung gemacht hätte, und dann eben den Speicher hier immer freigibt.

Besser, aber immer noch im Sinne des Erfinders wäre vielleicht:

static const char default_conf_filename[] = DEFAULT_CONF_FILENAME;
...
#ifdef XML_CONFIG
    if (opts->conf_filename && opts->conf_filename != default_conf_filename) {
            s_free(opts->conf_filename);
    }
    opts->conf_filename = default_conf_filename;
#endif

Liebe Grüße,
Axel