Check a return value from 'g_stat()'
authorJohn Emmas <johne53@tiscali.co.uk>
Wed, 14 Sep 2016 08:57:11 +0000 (09:57 +0100)
committerJohn Emmas <johne53@tiscali.co.uk>
Wed, 14 Sep 2016 08:58:10 +0000 (09:58 +0100)
commit83b48f5cc4b2ffcfdba12b34dd3b6f5ee67c8ca4
tree8224e52104059ca85462ada232b384bd53fd83f5
parent3b7cb8275a447773b48bea093bdc801c3f484ad6
Check a return value from 'g_stat()'

Some Mixbus users (on Windows) have reported seeing ludicrously high figures for the amount of disk space that'll be recovered if they choose to clean up unused sources. I can't see anything obviously wrong in Ardour's code - except for one situation where we don't check a return value after calling 'g_stat()'.

On Windows, the relevant path should be (hopefully!) in UTF8 format and the first thing that g_stat() does is to convert it to UTF16. If that conversion fails for some reason, g_stat() will return an error status and statbuf will be uninitialized - but at the moment, we're not checking this. As an experiment, let's check the returned value and find out if these user reports go away.

Unfortunately, if it does fix the problem then we've got an even bigger problem - because somehow, a Windows user can create source files with invalid names which can't be later deleted!!
libs/ardour/session_state.cc