openbsd/src/lib/libc/stdlib/realpath.c, branch OPENBSD_6_8_BASE A mirror of https://github.com/libressl/openbsd.git https://git.lua4.win/openbsd/atom?h=OPENBSD_6_8_BASE 2019-07-05T05:04:26+00:00 The last consumer of pre-posix realpath behaviour has stopped 2019-07-05T05:04:26+00:00 deraadt 2019-07-05T05:04:26+00:00 urn:sha1:cf5049b76fb3fdb0cbabcd4bf2d38f55144d1ee2 requiring it (sftp-server). Remove the /exists///// behaviour from here. The /nonexistant behaviour remains in the kernel and needs to be shot next. There may be ports fallout, but we doubt it. ok beck djm Remove old realpath(3), and the userland-vs-kernel realpath verification 2019-06-17T03:13:17+00:00 deraadt 2019-06-17T03:13:17+00:00 urn:sha1:f4e9d676b872828ec2961a190bb3647126844317 code. We now user the simple userland wrapper on top of __realpath(2). The non-POSIX behaviour still remains, that is the next component to fix. From a diff by beck, but I managed to chainsaw it a bit further. Tested in snaps for a couple of days. ok beck __realpath(2) appears to have improved, so re-enable the code that 2019-05-30T13:22:48+00:00 deraadt 2019-05-30T13:22:48+00:00 urn:sha1:cb1e90dd877fb015ed84ee7e9d87092087c31e09 checks userland-parsing vs kernel parsing, we are hoping to spot another bug.. There are some bugs in __realpath(2) -- it isn't quite ready so disable 2019-05-29T11:54:49+00:00 deraadt 2019-05-29T11:54:49+00:00 urn:sha1:9a36c0b50f6041d0ea3f74fbf3421a930cf9eb9f calling it until those are fixed. Enable the use of the kernel __realpath() system call in the libc wrapper. 2019-05-28T13:08:56+00:00 beck 2019-05-28T13:08:56+00:00 urn:sha1:4872fb7c56d08e3ef3a09ea7a8a8b91b85df09f7 For now, this also still uses the existing realpath implmentation and emits a syslog if we see differening results. Once we have run with that for a little while we will remove the old code ok deraadt@ Fix one possible buffer overflow and one underflow. Also some minor 2017-12-24T01:50:50+00:00 millert 2017-12-24T01:50:50+00:00 urn:sha1:09a3e37879c0ce6771d707ed7e8cca347ff20701 cleanups. From Jan Kokemueller. OK deraadt@ Don't call lstat() before readlink() just to see if it's a symlink, 2016-08-28T04:08:59+00:00 guenther 2016-08-28T04:08:59+00:00 urn:sha1:cc903521436dc1beb0550665c52b644138fdb51d as readlink() will tell you that more cheaply. ok millert@ In rev 1.15 the sizeof argument was fixed in a strlcat() call but 2015-10-13T20:55:37+00:00 millert 2015-10-13T20:55:37+00:00 urn:sha1:96da573f99f59914fdb5787f3b7529300fea564b the truncation check immediately following it was not updated to match. Not an issue in practice since the buffers are the same size. OK deraadt@ Move to the <limits.h> universe. 2015-01-16T16:48:51+00:00 deraadt 2015-01-16T16:48:51+00:00 urn:sha1:7027596ae70e4cf09e00ec2f7cb7434e59c533db review by millert, binary checking process with doug, concept with guenther Revert last commit due to changed semantics found by make release. 2014-10-19T03:56:28+00:00 doug 2014-10-19T03:56:28+00:00 urn:sha1:499633ffdbf1859969f1b13595936c5b64cc82e0