On Tuesday, 22 March 2005 at 14:04, Tamotsu Takahashi wrote: > BTW, What about checking the return value of sscanf? > (as Yonetani-san said.) > > if (sscanf (pc, "%lu", &(idata->uid_validity)) != 1) > goto fail; probably a good idea in theory, but next thing you know we're checking the return value of malloc calls.