On Wed, Feb 4, 2015 at 1:52 PM, Carl Worth <cworth at cworth.org> wrote: > - snprintf(cache->path, sizeof cache->path, > - "%s/.cache/mesa", pwd.pw_dir); > + cache->path = strdup(path); > + if (cache->path == NULL) { > + free (cache); No space after the function name. > + cache = NULL; > + goto done; > + }