On Tue, 2010-02-02 at 15:25 -0500, Steve Lawrence wrote: > + const char * rootDir = rpmtsRootDir(ts); > + int dochroot = (rootDir != NULL && !rstreq(rootDir, "/") && *rootDir == '/'); > + if (dochroot) { Wouldn't this be better as: int dochroot = (rootDir != NULL && rootDir[0] == '/' && rootDir[1]);