[Poppler-bugs] [Bug 67666] Correction for DEBUG_MEM usage

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Tue Aug 6 03:07:31 PDT 2013


https://bugs.freedesktop.org/show_bug.cgi?id=67666

Thomas Freitag <Thomas.Freitag at alfa.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #83532|0                           |1
        is obsolete|                            |

--- Comment #7 from Thomas Freitag <Thomas.Freitag at alfa.de> ---
Created attachment 83708
  --> https://bugs.freedesktop.org/attachment.cgi?id=83708&action=edit
use copyString where memory is freed with gfree

(In reply to comment #6)
> You need to improve your grepping skills ;-)

Upps, grep copyString */*.h is not the best way to find all occurences of
copyString :-)


> copyString is used in loooooooots of files, not only Object.h
> 
> Since it's actually an xpdf function i'd prefer if we use that to keep
> merging with a future possible xpdf 3.04 easier.
> 
> Do you want to upload a patch with copyString for those places or shall I?

Here it is. I just replace the call of strdup with copyString where the pointer
is freed with gfree().

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler-bugs/attachments/20130806/76afefa4/attachment.html>


More information about the Poppler-bugs mailing list