diff options
| author | Alexander Færøy <ahf@torproject.org> | 2017-03-08 23:11:42 +0100 |
|---|---|---|
| committer | Alexander Færøy <ahf@torproject.org> | 2017-03-09 00:10:18 +0100 |
| commit | 02fc0a5ecfc917e9261596816e926468db976453 (patch) | |
| tree | 8d8c32a5d77138d12843bbde26302b6d6b16c603 /src/common/compat.h | |
| parent | 02ef06516e64c1559b24123d7c7d164b76110c9a (diff) | |
| download | tor-02fc0a5ecfc917e9261596816e926468db976453.tar.gz tor-02fc0a5ecfc917e9261596816e926468db976453.zip | |
Remove fgets() compatbility function and related tests.
This patch removes the `tor_fgets()` wrapper around `fgets(3)` since it
is no longer needed. The function was created due to inconsistency
between the returned values of `fgets(3)` on different versions of Unix
when using `fgets(3)` on non-blocking file descriptors, but with the
recent changes in bug #21654 we switch from unbuffered to direct I/O on
non-blocking file descriptors in our utility module.
We continue to use `fgets(3)` directly in the geoip and dirserv module
since this usage is considered safe.
This patch also removes the test-case that was created to detect
differences in the implementation of `fgets(3)` as well as the changes
file since these changes was not included in any releases yet.
See: https://bugs.torproject.org/21654
Diffstat (limited to 'src/common/compat.h')
| -rw-r--r-- | src/common/compat.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/common/compat.h b/src/common/compat.h index 1f51ece61f..ee1c9454de 100644 --- a/src/common/compat.h +++ b/src/common/compat.h @@ -740,8 +740,6 @@ STATIC int tor_ersatz_socketpair(int family, int type, int protocol, ssize_t tor_getpass(const char *prompt, char *output, size_t buflen); -char *tor_fgets(char *str, int size, FILE *stream); - /* This needs some of the declarations above so we include it here. */ #include "compat_threads.h" |