This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Committing patches without review (Re: GNU C Library master sources branch master updated. glibc-2.19-812-g7ee03f0)
- From: Siddhesh Poyarekar <siddhesh dot poyarekar at gmail dot com>
- To: meyering at sourceware dot org
- Cc: GNU C Library <libc-alpha at sourceware dot org>
- Date: Mon, 14 Jul 2014 17:05:27 +0530
- Subject: Committing patches without review (Re: GNU C Library master sources branch master updated. glibc-2.19-812-g7ee03f0)
- Authentication-results: sourceware.org; auth=none
Hi Jim,
I noticed that you committed this patch earlier today; that's not how
we do it in glibc nowadays. There needs to be a review and ack by at
least another maintainer before a patch makes it in. Some people have
blanket commit access to parts of the repo, but even they ought to be
posting a note on libc-alpha with the patch to notify others of the
change.
As for the change itself, it seems fine to me, but please also mention
the fixed bug in NEWS and close the bug.
Thanks,
Siddhesh
[1] https://sourceware.org/glibc/wiki/MAINTAINERS
On 14 July 2014 09:56, <meyering@sourceware.org> wrote:
> This is an automated email from the git hooks/post-receive script. It was
> generated because a ref change was pushed to the repository containing
> the project "GNU C Library master sources".
>
> The branch, master has been updated
> via 7ee03f00188723a4de2b85021e511ced6d7fc4be (commit)
> from 9d39d517a7c700d53e2f24e8855a9b3a5e046a63 (commit)
>
> Those revisions listed above that are new to this repository have
> not appeared on any other notification email; so we list those
> revisions in full, below.
>
> - Log -----------------------------------------------------------------
> http://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=7ee03f00188723a4de2b85021e511ced6d7fc4be
>
> commit 7ee03f00188723a4de2b85021e511ced6d7fc4be
> Author: Jim Meyering <meyering@fb.com>
> Date: Sat Jul 12 17:07:21 2014 -0700
>
> regex: don't deref NULL upon heap allocation failure
>
> * posix/regcomp.c: (parse_dup_op): Handle duplicate_tree
> failure in one more place.
> To trigger the segfault, configure grep -with-included-regex,
> build it, and run these commands:
> ( ulimit -v 300000; echo a|src/grep -E a+++++++++++++++++++++ )
>
> diff --git a/ChangeLog b/ChangeLog
> index 3dd3cf0..41247ec 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,3 +1,13 @@
> +2014-07-13 Jim Meyering <meyering@fb.com>
> +
> + [BZ 17150]
> + regex: don't deref NULL upon heap allocation failure
> + * posix/regcomp.c: (parse_dup_op): Handle duplicate_tree
> + failure in one more place.
> + To trigger the segfault, configure grep -with-included-regex,
> + build it, and run these commands:
> + ( ulimit -v 300000; echo a|src/grep -E a+++++++++++++++++++++ )
> +
> 2014-07-13 Andreas Schwab <schwab@linux-m68k.org>
>
> * sysdeps/m68k/m680x0/fpu/libm-test-ulps: Update.
> diff --git a/posix/regcomp.c b/posix/regcomp.c
> index 076eca3..8f2747b 100644
> --- a/posix/regcomp.c
> +++ b/posix/regcomp.c
> @@ -2582,6 +2582,8 @@ parse_dup_op (bin_tree_t *elem, re_string_t *regexp, re_dfa_t *dfa,
>
> /* Duplicate ELEM before it is marked optional. */
> elem = duplicate_tree (elem, dfa);
> + if (BE (elem == NULL, 0))
> + goto parse_dup_op_espace;
> old_tree = tree;
> }
> else
>
> -----------------------------------------------------------------------
>
> Summary of changes:
> ChangeLog | 10 ++++++++++
> posix/regcomp.c | 2 ++
> 2 files changed, 12 insertions(+), 0 deletions(-)
>
>
> hooks/post-receive
> --
> GNU C Library master sources
--
http://siddhesh.in