aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--NEWS6
-rw-r--r--src/archive/bz2_plugin.c5
-rw-r--r--src/archive/iso_plugin.c5
-rw-r--r--src/archive/zip_plugin.c5
-rw-r--r--src/archive_plugin.h3
-rw-r--r--src/input/archive_input_plugin.c1
-rw-r--r--test/run_input.c105
7 files changed, 85 insertions, 45 deletions
diff --git a/NEWS b/NEWS
index 2747a781b..32aec7004 100644
--- a/NEWS
+++ b/NEWS
@@ -79,6 +79,12 @@ ver 0.16 (20??/??/??)
ver 0.15.7 (2009/??/??)
+* archive:
+ - close archive when stream is closed
+ - iso, zip: fixed memory leak in destructor
+* input:
+ - file: don't fall back to parent directory
+ - archive: fixed memory leak in error handler
* tags:
- id3: fix ID3v1 charset conversion
* decoders:
diff --git a/src/archive/bz2_plugin.c b/src/archive/bz2_plugin.c
index 20f8a427b..693dd4dba 100644
--- a/src/archive/bz2_plugin.c
+++ b/src/archive/bz2_plugin.c
@@ -144,8 +144,7 @@ bz2_close(struct archive_file *file)
{
bz2_context *context = (bz2_context *) file;
- if (context->name)
- g_free(context->name);
+ g_free(context->name);
input_stream_close(&context->istream);
g_free(context);
@@ -179,6 +178,8 @@ bz2_is_close(struct input_stream *is)
bz2_context *context = (bz2_context *) is->data;
bz2_destroy(context);
is->data = NULL;
+
+ bz2_close((struct archive_file *)context);
}
static bool
diff --git a/src/archive/iso_plugin.c b/src/archive/iso_plugin.c
index 04e0cfb3f..b56653d56 100644
--- a/src/archive/iso_plugin.c
+++ b/src/archive/iso_plugin.c
@@ -133,7 +133,8 @@ iso_close(struct archive_file *file)
}
//close archive
iso9660_close(context->iso);
- context->iso = NULL;
+
+ g_free(context);
}
/* single archive handling */
@@ -166,6 +167,8 @@ iso_is_close(struct input_stream *is)
{
iso_context *context = (iso_context *) is->data;
g_free(context->statbuf);
+
+ iso_close((struct archive_file *)context);
}
diff --git a/src/archive/zip_plugin.c b/src/archive/zip_plugin.c
index 9078c2d32..95bc1d02b 100644
--- a/src/archive/zip_plugin.c
+++ b/src/archive/zip_plugin.c
@@ -100,7 +100,8 @@ zip_close(struct archive_file *file)
}
//close archive
zzip_dir_close (context->dir);
- context->dir = NULL;
+
+ g_free(context);
}
/* single archive handling */
@@ -134,6 +135,8 @@ zip_is_close(struct input_stream *is)
{
zip_context *context = (zip_context *) is->data;
zzip_file_close (context->file);
+
+ zip_close((struct archive_file *)context);
}
static size_t
diff --git a/src/archive_plugin.h b/src/archive_plugin.h
index 92310f01a..df2dcff47 100644
--- a/src/archive_plugin.h
+++ b/src/archive_plugin.h
@@ -65,6 +65,9 @@ struct archive_plugin {
/**
* Opens an input_stream of a file within the archive.
*
+ * If this function succeeds, then the #input_stream "owns"
+ * the archive file and will automatically close it.
+ *
* @param path the path within the archive
*/
bool (*open_stream)(struct archive_file *, struct input_stream *is,
diff --git a/src/input/archive_input_plugin.c b/src/input/archive_input_plugin.c
index a98bd9e2a..0de715a82 100644
--- a/src/input/archive_input_plugin.c
+++ b/src/input/archive_input_plugin.c
@@ -67,6 +67,7 @@ input_archive_open(struct input_stream *is, const char *pathname)
if (!opened) {
g_warning("open inarchive file %s failed\n\n",filename);
+ arplug->close(file);
} else {
is->ready = true;
}
diff --git a/test/run_input.c b/test/run_input.c
index 88202063f..bd66a1b1a 100644
--- a/test/run_input.c
+++ b/test/run_input.c
@@ -24,6 +24,10 @@
#include "tag_save.h"
#include "conf.h"
+#ifdef ENABLE_ARCHIVE
+#include "archive_list.h"
+#endif
+
#include <glib.h>
#include <unistd.h>
@@ -38,46 +42,17 @@ my_log_func(const gchar *log_domain, G_GNUC_UNUSED GLogLevelFlags log_level,
g_printerr("%s\n", message);
}
-int main(int argc, char **argv)
+static int
+dump_input_stream(struct input_stream *is)
{
- struct input_stream is;
- GError *error = NULL;
- bool success;
char buffer[4096];
size_t num_read;
ssize_t num_written;
- if (argc != 2) {
- g_printerr("Usage: run_input URI\n");
- return 1;
- }
+ /* wait until the stream becomes ready */
- /* initialize GLib */
-
- g_thread_init(NULL);
- g_log_set_default_handler(my_log_func, NULL);
-
- /* initialize MPD */
-
- tag_pool_init();
- config_global_init();
-
- if (!input_stream_global_init(&error)) {
- g_warning("%s", error->message);
- g_error_free(error);
- return 2;
- }
-
- /* open the stream and wait until it becomes ready */
-
- success = input_stream_open(&is, argv[1]);
- if (!success) {
- g_printerr("input_stream_open() failed\n");
- return 2;
- }
-
- while (!is.ready) {
- int ret = input_stream_buffer(&is);
+ while (!is->ready) {
+ int ret = input_stream_buffer(is);
if (ret < 0)
/* error */
return 2;
@@ -89,20 +64,20 @@ int main(int argc, char **argv)
/* print meta data */
- if (is.mime != NULL)
- g_printerr("MIME type: %s\n", is.mime);
+ if (is->mime != NULL)
+ g_printerr("MIME type: %s\n", is->mime);
/* read data and tags from the stream */
- while (!input_stream_eof(&is)) {
- struct tag *tag = input_stream_tag(&is);
+ while (!input_stream_eof(is)) {
+ struct tag *tag = input_stream_tag(is);
if (tag != NULL) {
g_printerr("Received a tag:\n");
tag_save(stderr, tag);
tag_free(tag);
}
- num_read = input_stream_read(&is, buffer, sizeof(buffer));
+ num_read = input_stream_read(is, buffer, sizeof(buffer));
if (num_read == 0)
break;
@@ -111,12 +86,60 @@ int main(int argc, char **argv)
break;
}
+ return 0;
+}
+
+int main(int argc, char **argv)
+{
+ GError *error = NULL;
+ struct input_stream is;
+ int ret;
+
+ if (argc != 2) {
+ g_printerr("Usage: run_input URI\n");
+ return 1;
+ }
+
+ /* initialize GLib */
+
+ g_thread_init(NULL);
+ g_log_set_default_handler(my_log_func, NULL);
+
+ /* initialize MPD */
+
+ tag_pool_init();
+ config_global_init();
+
+#ifdef ENABLE_ARCHIVE
+ archive_plugin_init_all();
+#endif
+
+ if (!input_stream_global_init(&error)) {
+ g_warning("%s", error->message);
+ g_error_free(error);
+ return 2;
+ }
+
+ /* open the stream and dump it */
+
+ if (input_stream_open(&is, argv[1])) {
+ ret = dump_input_stream(&is);
+ input_stream_close(&is);
+ } else {
+ g_printerr("input_stream_open() failed\n");
+ ret = 2;
+ }
+
/* deinitialize everything */
- input_stream_close(&is);
input_stream_global_finish();
+
+#ifdef ENABLE_ARCHIVE
+ archive_plugin_deinit_all();
+#endif
+
config_global_finish();
tag_pool_deinit();
- return 0;
+ return ret;
}