mpool/mpool_get.c: Avoid clobbering 'errno' when handling 'pread' errors

[bnovkov]

Summary

POSIX.1-2024 states that the 'free' function "shall not modify errno if
ptr is a null pointer or a pointer previously returned as if by malloc()
and not yet deallocated". However this is a fairly recent addition
and non-compliant allocators might still clobber 'errno', causing
'mpool_get' to return the wrong error code. Fix this by saving
and restoring 'errno' after calling 'free'.

Sponsored by: Klara, Inc.

[obiwac]

lib/libc/db/mpool/mpool.c

236–241

here there's not really a point in saving errno if we're going to set it to EINVAL anyways

[bnovkov]

Remove redundant errno save

[bnovkov]

lib/libc/db/mpool/mpool.c

236–241

whoops, pattern-matched a bit too greedily, thanks!