[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v3 05/10] libxl: localtime(3) can return NULL



Wei Liu writes ("[PATCH v3 05/10] libxl: localtime(3) can return NULL"):
> Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
...
>          t = time(NULL);
>          tm = localtime(&t);
>  
> +        if (!tm) {
> +            ret = ERROR_FAIL;

You must log something here I think.  Otherwise this function
generates ERROR_FAIL entirely mysteriously (in this quite unlikely
error path).

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.