* [Buildroot] [PATCH] diffutils stdio: don't assume gets any more
@ 2012-11-09 13:02 Alexander Khryukin
2012-11-10 1:02 ` Arnout Vandecappelle
0 siblings, 1 reply; 2+ messages in thread
From: Alexander Khryukin @ 2012-11-09 13:02 UTC (permalink / raw)
To: buildroot
Signed-off-by: Alexander Khryukin <alexander@mezon.ru>
---
package/diffutils/diffutils-3.2-no-gets.patch | 21 +++++++++++++++++++++
1 file changed, 21 insertions(+)
create mode 100644 package/diffutils/diffutils-3.2-no-gets.patch
diff --git a/package/diffutils/diffutils-3.2-no-gets.patch b/package/diffutils/diffutils-3.2-no-gets.patch
new file mode 100644
index 0000000..56ca22d
--- /dev/null
+++ b/package/diffutils/diffutils-3.2-no-gets.patch
@@ -0,0 +1,21 @@
+hack until gzip pulls a newer gnulib version
+
+From 66712c23388e93e5c518ebc8515140fa0c807348 Mon Sep 17 00:00:00 2001
+From: Eric Blake <eblake@redhat.com>
+Date: Thu, 29 Mar 2012 13:30:41 -0600
+Subject: [PATCH] stdio: don't assume gets any more
+
+Gnulib intentionally does not have a gets module, and now that C11
+and glibc have dropped it, we should be more proactive about warning
+any user on a platform that still has a declaration of this dangerous
+interface.
+
+--- a/lib/stdio.in.h
++++ b/lib/stdio.in.h
+@@ -125,7 +125,6 @@
+ /* It is very rare that the developer ever has full control of stdin,
+ so any use of gets warrants an unconditional warning. Assume it is
+ always declared, since it is required by C89. */
+-_GL_WARN_ON_USE (gets, "gets is a security hole - use fgets instead");
+ #endif
+
--
1.7.11.3
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [Buildroot] [PATCH] diffutils stdio: don't assume gets any more
2012-11-09 13:02 [Buildroot] [PATCH] diffutils stdio: don't assume gets any more Alexander Khryukin
@ 2012-11-10 1:02 ` Arnout Vandecappelle
0 siblings, 0 replies; 2+ messages in thread
From: Arnout Vandecappelle @ 2012-11-10 1:02 UTC (permalink / raw)
To: buildroot
Change the commit message into:
diffutils: fix build with glibc2.16, which doesn't declare gets()
On 11/09/12 14:02, Alexander Khryukin wrote:
>
> Signed-off-by: Alexander Khryukin<alexander@mezon.ru>
> ---
> package/diffutils/diffutils-3.2-no-gets.patch | 21 +++++++++++++++++++++
> 1 file changed, 21 insertions(+)
> create mode 100644 package/diffutils/diffutils-3.2-no-gets.patch
>
> diff --git a/package/diffutils/diffutils-3.2-no-gets.patch b/package/diffutils/diffutils-3.2-no-gets.patch
> new file mode 100644
> index 0000000..56ca22d
> --- /dev/null
> +++ b/package/diffutils/diffutils-3.2-no-gets.patch
> @@ -0,0 +1,21 @@
> +hack until gzip pulls a newer gnulib version
> +
You should still add your SoB here, to indicate that you were in the
forwarding path.
Even so,
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Regards,
Arnout
[snip]
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286540
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-11-10 1:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-09 13:02 [Buildroot] [PATCH] diffutils stdio: don't assume gets any more Alexander Khryukin
2012-11-10 1:02 ` Arnout Vandecappelle
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox