Skip to content
This repository was archived by the owner on Nov 8, 2023. It is now read-only.

Commit 36a37f0

Browse files
Treehugger RobotGerrit Code Review
authored andcommitted
Merge "status.md: actually write down what's wrong with wordexp()." into main
2 parents 3b3786a + b115a8c commit 36a37f0

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

docs/status.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,14 @@ Missing functions are either obsolete or explicitly disallowed by SELinux:
3232
* `ualarm`
3333

3434
Missing functionality:
35-
* `<aio.h>`
35+
* `<aio.h>`. No particular reason not to have this other than that no-one's
36+
needed it yet, and it's relatively complex. If/when llvm-libc adds this,
37+
maybe we'll just reuse that.
3638
* `<monetary.h>`. See
3739
[discussion](https://github.com/android/ndk/issues/1182).
38-
* `<wordexp.h>`
40+
* `<wordexp.h>`. Unsafe because it passes user input to the shell (!),
41+
and often should just be a call to glob() anyway. See also
42+
[OpenBSD's discussion about adding wordexp()](https://www.mail-archive.com/[email protected]/msg02325.html).
3943
* Locales. Although bionic contains the various `_l()` functions, the only
4044
locale supported is a UTF-8 C/POSIX locale. Most of the POSIX APIs are
4145
insufficient to support the wide range of languages used by Android users,

0 commit comments

Comments
 (0)