This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc 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: Fix BZ #18757 (fmemopen doesn't fail with invalid modes)

From: Paul Pluzhnikov <ppluzhnikov at gmail dot com>
To: Paul Pluzhnikov <ppluzhnikov at gmail dot com>, GLIBC Devel <libc-alpha at sourceware dot org>, Paul Pluzhnikov <ppluzhnikov at google dot com>
Date: Sun, 9 Aug 2015 22:00:51 -0700
Subject: Re: Fix BZ #18757 (fmemopen doesn't fail with invalid modes)
Authentication-results: sourceware.org; auth=none
References: <CALoOobOhaK-1ZU4yJL7UZdgH1wqT-_oQ1Kx4Qhe8ENdG-=_yfg at mail dot gmail dot com> <20150810023644 dot GD23550 at vapier> <CALoOobNGvmPXiAGuL5XfeUaf-ueMB-Y7NvtEWgFPpVTv0EW=vA at mail dot gmail dot com> <20150810041505 dot GH23550 at vapier>

On Sun, Aug 9, 2015 at 9:15 PM, Mike Frysinger <vapier@gentoo.org> wrote:

> i think you also need to free c->buffer when c->mybuffer is true

Right.


2015-08-09  Paul Pluzhnikov  <ppluzhnikov@google.com>

        [BZ #18757]
        * libio/fmemopen.c (__fmemopen): Set EINVAL and don't leak memory.
        * libio/oldfmemopen.c (__old_fmemopen): Likewise.
        * libio/test-fmemopen.c (do_bz18757): New test.

-- 
Paul Pluzhnikov

Attachment: bz18757-20150809b.txt
Description: Text document

Follow-Ups:
- Re: Fix BZ #18757 (fmemopen doesn't fail with invalid modes)
  - From: Mike Frysinger

References:
- Fix BZ #18757 (fmemopen doesn't fail with invalid modes)
  - From: Paul Pluzhnikov
- Re: Fix BZ #18757 (fmemopen doesn't fail with invalid modes)
  - From: Mike Frysinger
- Re: Fix BZ #18757 (fmemopen doesn't fail with invalid modes)
  - From: Paul Pluzhnikov
- Re: Fix BZ #18757 (fmemopen doesn't fail with invalid modes)
  - From: Mike Frysinger

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