Hi Michael,
> injection_points: Add injection_points_list()
>
> This function can be used to retrieve the information about all the
> injection points attached to a cluster, providing coverage for
> InjectionPointList() introduced in 7b2eb72b1b8c.
>
> The original proposal turned around a system function, but that would
> not be backpatchable to stable branches. It was also a bit weird to
> have a system function that fails depending on if the build allows
> injection points or not.
In injection_points_list() isn't `memset(values, 0, sizeof(values))`
redundant? Also perhaps we want to pfree inj_points.
These are minor issues of course.