On Fri, Nov 25, 2011 at 2:41 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: > It's valid to pass NULL GError ** around when we are not interested > in error reporting. This means we have to make sure the GError** is > not NULL before dereferencing it. In particular, the right way of > testing if GError **err; is set is to do if (err && *err) {} > This commit adds a helper to do this check and uses it where needed. Looks correct! Ack! -- Regards, Zeeshan Ali (Khattak) FSF member#5124