That was my point yes, that it is a ProgrammingError to call
SetPolicyServer(NULL). Hopefully we don't do it somewhere else, if we do
the assert() will catch it in debug builds.


On Mon, Jun 3, 2013 at 10:08 AM, Kristian Amlie <notifications@github.com>wrote:

> In libpromises/generic_agent.c:
>
> >          }
> > -        SetPolicyServer(ctx, existing_policy_server);
>
> Right, of course. I meant empty string. Just asking because I'm unsure
> what this fixes. The last commit does not seem to change anything; it will
> still be empty string, like it was before. The only difference is the
> assert. But maybe that's the point?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cfengine/core/pull/725/files#r4495381>
> .
>