Fwd: [PATCHES] simple gpo patches fixes

[Douglas Bagnall]

On 14/04/18 02:19, David Mulder via samba-technical wrote:
> I've added that type check, as well as couple other checks. I also
> wasn't checking the ads type input or verifying the list and it's length
> before popping from it. Latest sources are on the github pull request.

Thanks. The ADS check is definitely necessary.

I thought the list was OK without the checks (in that you'd get a Python
exception from PyList_GetItem(), not a segfault), but I see that you're
accepting an empty list, so you do need more checks.

Two questions:

1. Is there a test for the empty list?

2. If you're only ever looking at the first item, why is it a list?

The first question is a "there should be" hint, but the second one is a
genuine question from a GPO ignoramus.

Douglas