#include "curl_download.h"
#include "preferences.h"
#include "globals.h"
+#include "vik_compat.h"
static gboolean check_file_first_line(FILE* f, gchar *patterns[])
{
size_t nr;
memset(buf, 0, sizeof(buf));
- fgetpos(f, &pos);
+ if ( !fgetpos(f, &pos) )
+ return FALSE;
rewind(f);
nr = fread(buf, 1, sizeof(buf) - 1, f);
- fsetpos(f, &pos);
+ if ( !fgetpos(f, &pos) )
+ return FALSE;
for (bp = buf; (bp < (buf + sizeof(buf) - 1)) && (nr > (bp - buf)); bp++) {
if (!(isspace(*bp)))
break;
VikLayerParamData tmp;
tmp.u = VIK_CONFIG_DEFAULT_TILE_AGE / 86400; // Now in days
a_preferences_register(prefs, tmp, VIKING_PREFERENCES_GROUP_KEY);
+ file_list_mutex = vik_mutex_new();
+}
- file_list_mutex = g_mutex_new();
+void a_download_uninit (void)
+{
+ vik_mutex_free(file_list_mutex);
}
static gboolean lock_file(const char *fn)
g_mutex_unlock(file_list_mutex);
}
-
+/**
+ * Unzip a file - replacing the file with the unzipped contents of the self
+ */
static void uncompress_zip ( gchar *name )
{
GError *error = NULL;
return;
}
- // This overwrires any previous file contents
+ // This overwrites any previous file contents
if ( ! g_file_set_contents ( name, unzip_mem, ucsize, &error ) ) {
g_critical ( "Couldn't write file '%s', because of %s", name, error->message );
g_error_free ( error );
void a_try_decompress_file (gchar *name)
{
#ifdef HAVE_MAGIC_H
+#ifdef MAGIC_VERSION
+ // Or magic_version() if available - probably need libmagic 5.18 or so
+ // (can't determine exactly which version the versioning became available)
+ g_debug ("%s: magic version: %d", __FUNCTION__, MAGIC_VERSION );
+#endif
magic_t myt = magic_open ( MAGIC_CONTINUE|MAGIC_ERROR|MAGIC_MIME );
gboolean zip = FALSE;
gboolean bzip2 = FALSE;
if ( myt ) {
- magic_load ( myt, NULL );
- const char* magic = magic_file (myt, name);
- g_debug ("%s: magic output: %s", __FUNCTION__, magic );
+#ifdef WINDOWS
+ // We have to 'package' the magic database ourselves :(
+ // --> %PROGRAM FILES%\Viking\magic.mgc
+ int ml = magic_load ( myt, ".\\magic.mgc" );
+#else
+ // Use system default
+ int ml = magic_load ( myt, NULL );
+#endif
+ if ( ml == 0 ) {
+ const char* magic = magic_file (myt, name);
+ g_debug ("%s: magic output: %s", __FUNCTION__, magic );
- if ( g_strcmp0 (magic, "application/zip; charset=binary") == 0 )
- zip = TRUE;
+ if ( g_strcmp0 (magic, "application/zip; charset=binary") == 0 )
+ zip = TRUE;
- if ( g_strcmp0 (magic, "application/x-bzip2; charset=binary") == 0 )
- bzip2 = TRUE;
+ if ( g_strcmp0 (magic, "application/x-bzip2; charset=binary") == 0 )
+ bzip2 = TRUE;
+ }
+ else {
+ g_critical ("%s: magic load database failure", __FUNCTION__ );
+ }
magic_close ( myt );
}
}
else if ( bzip2 ) {
gchar* bz2_name = uncompress_bzip2 ( name );
- g_remove ( name );
- g_rename ( bz2_name, name );
+ if ( bz2_name ) {
+ if ( g_remove ( name ) )
+ g_critical ("%s: remove file failed [%s]", __FUNCTION__, name );
+ if ( g_rename (bz2_name, name) )
+ g_critical ("%s: file rename failed [%s] to [%s]", __FUNCTION__, bz2_name, name );
+ }
}
return;
#endif
}
-static int download( const char *hostname, const char *uri, const char *fn, DownloadMapOptions *options, gboolean ftp, void *handle)
+static DownloadResult_t download( const char *hostname, const char *uri, const char *fn, DownloadMapOptions *options, gboolean ftp, void *handle)
{
FILE *f;
int ret;
if (options == NULL || (!options->check_file_server_time &&
!options->use_etag)) {
/* Nothing to do as file already exists and we don't want to check server */
- return -3;
+ return DOWNLOAD_NOT_REQUIRED;
}
time_t tile_age = a_preferences_get(VIKING_PREFERENCES_NAMESPACE "download_tile_age")->u;
/* Get the modified time of this file */
struct stat buf;
- g_stat ( fn, &buf );
+ (void)g_stat ( fn, &buf );
time_t file_time = buf.st_mtime;
if ( (time(NULL) - file_time) < tile_age ) {
/* File cache is too recent, so return */
- return -3;
+ return DOWNLOAD_NOT_REQUIRED;
}
if (options->check_file_server_time) {
} else {
gchar *dir = g_path_get_dirname ( fn );
- g_mkdir_with_parents ( dir , 0777 );
+ if ( g_mkdir_with_parents ( dir , 0777 ) != 0)
+ g_warning ("%s: Failed to mkdir %s", __FUNCTION__, dir );
g_free ( dir );
}
g_free ( tmpfilename );
if (options->use_etag)
g_free ( file_options.etag );
- return -4;
+ return DOWNLOAD_FILE_WRITE_ERROR;
}
f = g_fopen ( tmpfilename, "w+b" ); /* truncate file and open it */
if ( ! f ) {
g_free ( tmpfilename );
if (options->use_etag)
g_free ( file_options.etag );
- return -4;
+ return DOWNLOAD_FILE_WRITE_ERROR;
}
/* Call the backend function */
ret = curl_download_get_url ( hostname, uri, f, options, ftp, &file_options, handle );
- if (ret != DOWNLOAD_NO_ERROR && ret != DOWNLOAD_NO_NEWER_FILE) {
+ DownloadResult_t result = DOWNLOAD_SUCCESS;
+
+ if (ret != CURL_DOWNLOAD_NO_ERROR && ret != CURL_DOWNLOAD_NO_NEWER_FILE) {
g_debug("%s: download failed: curl_download_get_url=%d", __FUNCTION__, ret);
failure = TRUE;
+ result = DOWNLOAD_HTTP_ERROR;
}
if (!failure && options != NULL && options->check_file != NULL && ! options->check_file(f)) {
g_debug("%s: file content checking failed", __FUNCTION__);
failure = TRUE;
+ result = DOWNLOAD_CONTENT_ERROR;
}
fclose ( f );
g_free ( file_options.etag );
g_free ( file_options.new_etag );
}
- return -1;
+ return result;
}
if ( options->convert_file )
}
}
- if (ret == DOWNLOAD_NO_NEWER_FILE) {
- g_remove ( tmpfilename );
+ if (ret == CURL_DOWNLOAD_NO_NEWER_FILE) {
+ (void)g_remove ( tmpfilename );
#if GLIB_CHECK_VERSION(2,18,0)
g_utime ( fn, NULL ); /* update mtime of local copy */
#else
utimes ( fn, NULL ); /* update mtime of local copy */
#endif
} else {
- g_rename ( tmpfilename, fn ); /* move completely-downloaded file to permanent location */
+ /* move completely-downloaded file to permanent location */
+ if ( g_rename ( tmpfilename, fn ) )
+ g_warning ("%s: file rename failed [%s] to [%s]", __FUNCTION__, tmpfilename, fn );
}
unlock_file ( tmpfilename );
g_free ( tmpfilename );
g_free ( file_options.etag );
g_free ( file_options.new_etag );
}
- return 0;
+ return DOWNLOAD_SUCCESS;
}
-/* success = 0, -1 = couldn't connect, -2 HTTP error, -3 file exists, -4 couldn't write to file... */
-/* uri: like "/uri.html?whatever" */
-/* only reason for the "wrapper" is so we can do redirects. */
-int a_http_download_get_url ( const char *hostname, const char *uri, const char *fn, DownloadMapOptions *opt, void *handle )
+/**
+ * uri: like "/uri.html?whatever"
+ * only reason for the "wrapper" is so we can do redirects.
+ */
+DownloadResult_t a_http_download_get_url ( const char *hostname, const char *uri, const char *fn, DownloadMapOptions *opt, void *handle )
{
return download ( hostname, uri, fn, opt, FALSE, handle );
}
-int a_ftp_download_get_url ( const char *hostname, const char *uri, const char *fn, DownloadMapOptions *opt, void *handle )
+DownloadResult_t a_ftp_download_get_url ( const char *hostname, const char *uri, const char *fn, DownloadMapOptions *opt, void *handle )
{
return download ( hostname, uri, fn, opt, TRUE, handle );
}
}
tmp_file = fdopen(tmp_fd, "r+");
+ if ( !tmp_file )
+ return NULL;
if ( curl_download_uri ( uri, tmp_file, options, NULL, NULL ) ) {
// error
fclose ( tmp_file );
- g_remove ( tmpname );
+ (void)g_remove ( tmpname );
g_free ( tmpname );
return NULL;
}