about summary refs log tree commit diff
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2012-10-27 19:52:40 -0400
committerRich Felker <dalias@aerifal.cx>2012-10-27 19:52:40 -0400
commit8fc7b5965ac6a000c93c7362276a6a7b193647f4 (patch)
tree6ffe5b0141543fc39213a87290ec2e75a2540735
parent3ffb556c58ba69b4f52ae50fa92424300bd58bd1 (diff)
downloadmusl-8fc7b5965ac6a000c93c7362276a6a7b193647f4.tar.gz
musl-8fc7b5965ac6a000c93c7362276a6a7b193647f4.tar.xz
musl-8fc7b5965ac6a000c93c7362276a6a7b193647f4.zip
separate getc/putc from fgetc/fputc
for conformance, two functions should not have the same address. a
conforming program could use the addresses of getc and fgetc in ways
that assume they are distinct. normally i would just use a wrapper,
but these functions are so small and performance-critical that an
extra layer of function call could make the one that's a wrapper
nearly twice as slow, so I'm just duplicating the code instead.
-rw-r--r--src/stdio/fgetc.c3
-rw-r--r--src/stdio/fputc.c3
-rw-r--r--src/stdio/getc.c13
-rw-r--r--src/stdio/putc.c12
4 files changed, 25 insertions, 6 deletions
diff --git a/src/stdio/fgetc.c b/src/stdio/fgetc.c
index 65a10fa6..e1224164 100644
--- a/src/stdio/fgetc.c
+++ b/src/stdio/fgetc.c
@@ -9,6 +9,3 @@ int fgetc(FILE *f)
 	__unlockfile(f);
 	return c;
 }
-
-weak_alias(fgetc, getc);
-weak_alias(fgetc, _IO_getc);
diff --git a/src/stdio/fputc.c b/src/stdio/fputc.c
index 99532965..92762c98 100644
--- a/src/stdio/fputc.c
+++ b/src/stdio/fputc.c
@@ -8,6 +8,3 @@ int fputc(int c, FILE *f)
 	__unlockfile(f);
 	return c;
 }
-
-weak_alias(fputc, putc);
-weak_alias(fputc, _IO_putc);
diff --git a/src/stdio/getc.c b/src/stdio/getc.c
new file mode 100644
index 00000000..b3f351d1
--- /dev/null
+++ b/src/stdio/getc.c
@@ -0,0 +1,13 @@
+#include "stdio_impl.h"
+
+int getc(FILE *f)
+{
+	int c;
+	if (f->lock < 0 || !__lockfile(f))
+		return getc_unlocked(f);
+	c = getc_unlocked(f);
+	__unlockfile(f);
+	return c;
+}
+
+weak_alias(getc, _IO_getc);
diff --git a/src/stdio/putc.c b/src/stdio/putc.c
new file mode 100644
index 00000000..fa893496
--- /dev/null
+++ b/src/stdio/putc.c
@@ -0,0 +1,12 @@
+#include "stdio_impl.h"
+
+int putc(int c, FILE *f)
+{
+	if (f->lock < 0 || !__lockfile(f))
+		return putc_unlocked(c, f);
+	c = putc_unlocked(c, f);
+	__unlockfile(f);
+	return c;
+}
+
+weak_alias(putc, _IO_putc);