On Thu, 11.07.13 13:37, Dave Reisner (d...@falconindy.com) wrote:

> > +        p = strchr(path, ':');
> 
> This is going to fail for a file:value pair such as /foo:/bar/baz:value.
> You could use strrchr(), but then you have to be concerned about
> matching values with a colon.

This might become a problem, but then again, I think it is OK if some
files cannot be checked with this. I'd prefer using a space or "=" as
separator as a better choice though, as that's probably less frequent
than ":" in the names of files one would check with this condition
setting.

Lennart

-- 
Lennart Poettering - Red Hat, Inc.
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to