Liu Yuan <namei.unix at gmail.com> writes: > >> } else > >> + eprintf("failed to open %s: %m\n", path.buf); > > > need { for else clause? > > >> ret = SD_RES_UNKNOWN; Yes, quite right! Cheers, Chris.