diff options
| author |  Thiago Macieira <thiago@macieira.org> | 2014-08-17 10:42:47 -0700 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2014-08-17 12:01:56 -0600 |
| commit | f3d44db512f5e877065cbb0afdfd25ce206976dd (patch) | |
| tree | 62c39fcdd5ce14ec7c8e659c56f4bd2606645ac7 /save-html.c | |
| parent | 29f50c6aeeb31dfad7510d240e27905a9f62670d (diff) | |
| download | subsurface-f3d44db512f5e877065cbb0afdfd25ce206976dd.tar.gz | |
Fix and simplify get_file_name
This function created a QFile that it didn't need and it returned a
dangling pointer (the std::string object was destroyed at the end of the
function).
The function now returns a pointer that the caller must free. For that
reason, it's easier in copy_image_and_overwrite to just go for the
QFileInfo call.
Signed-off-by: Thiago Macieira <thiago@macieira.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'save-html.c')
| -rw-r--r-- | save-html.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/save-html.c b/save-html.c index 0288d447c..82d81edda 100644 --- a/save-html.c +++ b/save-html.c @@ -16,7 +16,9 @@ void save_photos(struct membuffer *b, const char *photos_dir, struct dive *dive) struct picture *pic = dive->picture_list; put_string(b, "\"photos\":["); while (pic) { - put_format(b, "{\"filename\":\"%s\"},", get_file_name(pic->filename)); + char *fname = get_file_name(pic->filename); + put_format(b, "{\"filename\":\"%s\"},", fname); + free(fname); copy_image_and_overwrite(pic->filename, photos_dir); pic = pic->next; } |