Bug 372478

Summary: utils/vk_utils.cpp:145]: (error) Memory leak: tmpname
Product: [Unmaintained] valkyrie Reporter: dcb314
Component: generalAssignee: Cerion Armour-Brown <cerion>
Status: RESOLVED UNMAINTAINED    
Severity: minor CC: philippe.waroquiers
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description dcb314 2016-11-14 18:55:19 UTC
Source code is

      char* tmpname = vk_str_malloc( unique.length() + 10 );
      sprintf( tmpname, "%s.XXXXXX", qPrintable( unique ) );
      int fd = mkstemp( tmpname );

      if ( fd == -1 ) {
         /* something went wrong */
         VK_DEBUG( "failed to create unique filename from '%s'.",
                   qPrintable( filepath ) );
         return QString::null;
      }
Comment 1 Philippe Waroquiers 2016-11-17 21:53:52 UTC
(In reply to dcb314 from comment #0)
> Source code is
> 
>       char* tmpname = vk_str_malloc( unique.length() + 10 );
>       sprintf( tmpname, "%s.XXXXXX", qPrintable( unique ) );
>       int fd = mkstemp( tmpname );
> 
>       if ( fd == -1 ) {
>          /* something went wrong */
>          VK_DEBUG( "failed to create unique filename from '%s'.",
>                    qPrintable( filepath ) );
>          return QString::null;
>       }

Are you sure this is for valgrind ?
I cannot find this vk_utils.cpp in the valgrind sources.
Comment 2 dcb314 2016-11-18 06:47:11 UTC
(In reply to Philippe Waroquiers from comment #1)
> Are you sure this is for valgrind ?
> I cannot find this vk_utils.cpp in the valgrind sources.

It's part of valkyrie. Is this not the right place for valkyrie bug reports ?
Comment 3 Philippe Waroquiers 2016-11-18 18:42:10 UTC
Changing the product to Valkyrie
Comment 4 Andrew Crouthamel 2018-09-04 16:06:13 UTC
Hello! Sorry to be the bearer of bad news, but this project has been unmaintained for many years so I am closing this bug.