On Sat, May 22, 2010 at 09:57:17PM +0200, Steinar H. Gunderson wrote: > + if (data) { > + *data = 1000; > + } > + if (type) { > + *type = REG_DWORD; > + } > + if (count) { > + *count = sizeof(DWORD); > + } Pointless {} > + free(c); > + return 0; Forgot to change that into a named constant.