Ok, committed (with the space fix). -- Caroline Tice cmt...@google.com
On Tue, Aug 20, 2013 at 12:43 PM, Florian Weimer <fwei...@redhat.com> wrote: > On 08/20/2013 09:15 PM, Caroline Tice wrote: >> >> That fixed it, thanks! Attached is the latest patch (Florian, I will >> send you the regenerated Makefile.in and configure separately). >> Please review and let me know if this is OK to commit! > > > As the libvtv reviewer, you don't need permission to commit your changes. > :-) > > But I bootstrapped on x86_64-debian-linux-gnu, and can confirm that this now > works as intended; __secure_getenv is picked up. > > One minor nit: > > Index: libvtv/vtv_utils.cc > =================================================================== > --- libvtv/vtv_utils.cc (revision 201802) > +++ libvtv/vtv_utils.cc (working copy) > > + decriptor. > > There's a stray space at the end of this line. > > > -- > Florian Weimer / Red Hat Product Security Team
