Bug 372478 - utils/vk_utils.cpp:145]: (error) Memory leak: tmpname
Summary: utils/vk_utils.cpp:145]: (error) Memory leak: tmpname
Status: RESOLVED UNMAINTAINED
Alias: None
Product: valkyrie
Classification: Unmaintained
Component: general (other bugs)
Version First Reported In: unspecified
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Cerion Armour-Brown
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-11-14 18:55 UTC by dcb314
Modified: 2018-09-04 16:06 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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.