PATCH: util-env

[Volker Lendecke]

"Trever L. Adams" <trever at middleearth.sapphiresunday.org> writes:

> On 10/12/2016 08:50 AM, Volker Lendecke wrote:
> +       if (data == NULL) return NULL;

We prefer

if (data == NULL) {
	return NULL;
}

with braces just for better debuggability and to avoid a "goto fail"
vulnerability.

Also, don't you have to terminate the env list with a NULL pointer?

Volker