zend_call_stack_get implementation for NetBSD.

[devnexen]

Despite being OpenBSD's predecessor, the approach is in fact a lot closer to Linux, at least in principle. We purposely avoid reading /proc/N/maps to be more future-proof.

[arnaud-lb]

Nice!

[devnexen]

ping :)

[arnaud-lb]

Sorry, I missed your update earlier.

This looks good to me, but I'm not a fan of the buf field in the zend_call_stack struct since it's a temporary. You could achieve the same result (preventing memory leaks) by extracting a part of zend_call_stack_get_netbsd_vm() in a separate function, like this:

zend_call_stack_get_netbsd_vm() {
    ...
    void *buf = malloc(...);
    zend_call_stack_get_netbsd_vm_read(...);
    free(buf);
    ...
}

[arnaud-lb]

Thank you!