Test crashes on Windows

[spanglerco]

When building the module locally on Windows, the program built from test.c crashes with a segfault. This is because the test is doing sizeof(void), which isn't valid in standard C and is implementation defined. GCC (by default) will convert that to 1, while MSVC will convert that to 0 and issue a warning C4034.

In the GCC case, it's only working because apr_pcalloc will align the allocation to 8 bytes, so it converts the 1 to an 8, which is large enough to fit a pointer. In the Windows case, apr_pcalloc with size 0 ends up returning a pointer to the next available free memory without actually reserving any memory, so request->server->module_config, request->per_dir_config, and whatever the next allocation is end up with the same pointer.

I see it was changed from sizeof(void *) to sizeof(void) in this commit, but I believe sizeof(void *) was correct, as ap_conf_vector_t * is really just a void **.

[zandbelt]

see 8a1dc8a