Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [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