apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <traw...@gmail.com>
Subject Re: Uninitialized 1 byte copy in apr_filepath_merge()
Date Thu, 08 Apr 2010 19:17:25 GMT
On Tue, Mar 30, 2010 at 4:37 AM, Arsen Chaloyan <achaloyan@yahoo.com> wrote:
> Hi,
>
> One small and pretty harmless, but still issue in the implementation of
> apr_filepath_merge() for win32.
>
> Wouldn't it be better to use apr_pstrmemdup() instead of apr_pmemdup() to
> avoid initialized 1 byte copy. At least, this allows to silence UMC errors
> memory profilers complain about.
>
> Index: file_io/win32/filepath.c
> ===================================================================

committed to trunk as r932067; I'll commit to 1.5.x and 1.4.x branches
if there are no complaints

thanks!

Mime
View raw message