This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH, vtv update] Fix /tmp directory issues in libvtv


Ok, committed (with the space fix).

-- Caroline Tice
cmtice@google.com

On Tue, Aug 20, 2013 at 12:43 PM, Florian Weimer <fweimer@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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]