Merge branch 'KMZ-Maps'

[DOC] KMZ Map help

KMZ file of OSM Mapnik rendering of Stonehenge area, zoom level 15.

[WINDOWS] Installer and build notes for using libzip.dll

Allow internal check_magic function to work on various lengths as specified.

Enable importing a map from a KMZ file.

Uses libzip to decompress the KMZ and processes the XML description to
 generate a corresponding GeoRef layer.

Allow creating GeoRef layers from a specified pixbuf and area.

Make the find best zoom level function reusable.

Add utility function to write a temporary file from a byte buffer.

Enable KMZ Map file generation for Custom Map support on Garmin devices.

The KMZ is a zipped file containing a KML file with the associated image.

Use libzip to generate multiple files in a single zip file.

Fix routes not saved in GPX when tracks are made invisible.

Probable copy and paste error in commit SHA:978cd6961f0cdfeb7d68fe18bc558011692cd7ce

SF Bugs#103: Fix TrackWaypoint layer items may not be displayed when pasted

When the target window has a different View mode,
 then the coordinates must be converted to match to be shown in the viewport.
This is already done for single item pastes, but previously not for entire TRW layers.

Enable Catalan and Turkish translations.

Seems these translations have been available for ages but not actually used.

Auto generate the date for man pages.

Restore opening of JPG files.

More lenient string comparison for magic checking of JPG files,
 as the text returned seems to have changed.

[QA] Tidy trw_layer_draw_point_names()

Remove unused variables and some minor whitespace improvements.

SF Bugs#127: Fix initial display of Waypoint sort order.

Ensure sort is applied on realization of the TrackWaypoint layer,
 otherwise it may not be performed.

Fix map layer widget sensitivity dependent on map type.

Merge pull request #8 from swegener/etag-in-xattr

Store etag in xattr

Merge pull request #3 from Shura0/point_names

Display trackpoint names

download: Move setting of etag and convert_file

It only makes sense to do this post-processing, if we got a new file, so
move the code just before the final rename.

Signed-off-by: Sven Wegener <sven.wegener@stealer.net>

download: Store ETAG in extended attributes

If supported, store the etag values from downloaded tiles in extended
attributes on the local files, instead of cluttering our maps directory
with .etag files.

Signed-off-by: Sven Wegener <sven.wegener@stealer.net>

Releasing Viking 1.6.1

Shift the reference scale to be generated from the centre of the display.

Previously this was done at the latitude of the bottom of the display.
For most zoomed in usage it makes no practical difference,
 but when zoomed out it's more logical as it's on average nearer to actual scale at locations across the visible map.

Update viking.doap file

[DOC] Words about various operations from the File menu not previously mentioned.

Ensure calculation of the waypoint bounds even when the layer is not currently visible.

Better reporting of failing to load any latlontz.txt file.

CID#34578: Remove logically dead code.

It's a repeat of the test above.
And explanation of when the test becomes true.

CID#132266: Fix potential dereference null return value

[QA] Clang-Analyzer: Don't store values that aren't subsequently used.

[QA] Clang-Analyzer - remove duplicate line

Ensure removing temporary file when it contains no wikipedia places.

[QA] Explicitly ignore return values when removing temporary generated files

[QA] CID#34606+CID#34607: Time of check time of use (TOCTOU) issues.

Mark as not security critical, use g_utime and check result.
Remove old GLIB version check as at least 2.26 must be used now.

Ensure memory is freed if errors are encountered in acquiring My OSM traces

CID#34579: Fix logically dead code.

This test is supposed to be for what the pointer is pointing to!

Coverity: Prevent deference after null checks

CID#34580
CID#34582
CID#34588
CID#34589

CID#34561+CID#34569: Check return values

CID#34586: Avoid potential explicit null deferenced

CID#34612: Fix incorrect break statement location.

CID#132255: Fix stray semicolon introduced by recent commit.

Fix broken commit in SHA:5e4cce8f309867bee2f16bc463e0f0b461d19ecf

CID#34592: Explicit handling of potential truncation of integer value

Coverity: Fix deference after null checks

CID#34585
CID#34590
CID#34591

CID#34600+CID#34601: Fix deference before null checks

[QA] CID#34594: Remove unsigned compared against 0 (as it has no effect)

Document previous translation updates

Import Launchpad translation updates

Include HACKING file in distribution.

Fix Georef mpp from coords.

Conversion from the image mpp to Viking zoom level is dependent on the latitude when in LATLON mode.

Make OSM services the default by including these modules first.

Simplify filter actually uses the unsigned integer value.

Fix Google maps positional web link.

Format seems to have changed recently.

[QA] Fix issue spotted with cppcheck 1.70

Remove unused northernHemisphere variable

[QA] Output console message if g_mkdir_with_parents() is unsuccessful.

CID#34603: Protection in copying into fixed size buffer

CID#34584+CID#34583: Prevent deference after null check

CID#34611+CID#34611: Prevent use of uninitialized scalar variables

Handle potentially unknown horiz_units, although this shouldn't normally happen

[QA] CID#34576: Remove logically dead code

[QA] CID#34575: Remove logically dead code.

Items are always considered for rename when moved.
The unique name can be straight copy of the original.

[QA] CID#34543: Adjust curl_get_etag_func to use pointer to structure rather than the raw address.

CID#131392: Avoid potential null pointer dereference

CID#34574: Fix freeing the appropriate variable.

[QA] Generally print warnings to the console for low level file operations
(since they don't normally go wrong)

CID#34548
CID#34549
CID#34550
CID#34552
CID#34558
CID#34560
CID#34562
CID#34564
CID#34568
CID#34569
CID#34571
CID#34572
CID#34573

[QA] CID#34547: Path value is checked so not worried about function return value.

[QA] Coverity: Unchecked return value from library

Explicitly ignore return values - when removing temporary generate files

CID#131383
CID#131382
CID#131381
CID#34570
CID#34563
CID#34561
CID#34557
CID#34555
CID#34546

Fix track tooltips minutes value should be rounded.

Thus need to ensure in floating point otherwise the minutes value gets truncated.

CID#101113: Result not floating point

Minutes value should be rounded, thus need to pass in as floating point.

[QA] CID#101114+CID#101115: Result not floating point

No need for round since the truncated integer division value is needed for the hours value.

[QA] CID#131384: Check function return value.

[QA] CID#131386: Check function result

[QA] CID#131385+CID#131387: Be more explicit that we don't care about the result of this function call.

[QA] CID#131393: Missing break in switch

[QA] CID#131394: Check result of g_fopen()

Add warnings if the LatLonTZ file can not be accessed or opened.

[QA] CID#131401+CID131400 - Improve printf arg types.

[QA] CID#131402: Fix resource leak in handling metatile errors.

Fix creating waypoints from JPGs when using gexiv2

SF Bugs#126: Fix crash in gdk_rgb_convert_0888 due to using deallocated memory.

There is reference counter in cached pixel buffers.
Previously it always equalled one although multiple tasks could be using it.
Thus when the cache is flushed, memory was always deallocated but then another task would attempt to use it and crash.

Thus now the cached pixel buffer is tracked properly with unref() after being used,
 so only when reference count is zero it is automatically deallocated.

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Disable Search and Routing with Google as they no longer support the APIs Viking used.

The only supported file formats seem to be JSON and XML in their own custom schemas (and not currently supported by GPSBabel).
Also now you need to supply an application key for all Google APIs.

Fix availability of Geotag consistency checks when using gexiv2

Remove --with-search configure options since they are not used.

They haven't been used since the search location web service engine was made selectable at run time back in 2010.

Improve place search failure message to say if it is a service level issue.

Expand vik_goto_tool_get_coord() to flow up if there is an error communicating with the web search provider.

Protect against possible out of bounds array access.

[QA] Coverity 'Uninitialized pointer read' CID#34608 + CID#34609.

Force initialise to NULL values, even if these variables get assigned from the function calls.

[QA] Coverity 'Buffer not null terminated' issues CID#34544 + CID#34545

Switch to use a function that ensures the buffer is null terminated.

Note that in these cases it is using a time format string that would always be less than the buffer size anyway.

Fix DEM height points on track graphs when height units is in feet and non-zero minimum height.

Offset is already in the current height units,
 so elevation value should be adjusted *after* it is in the appropriate units.

[DOC] Improved introduction.

Only perform man page generation when scrollkeeper is also installed.

[QA] Remove repeated statements in configure.ac

Fix mapnik preferences initialisation to not crash if preferences are not available.

Enable reference documentation generation by disabling optional components.

Otherwise at the linking stage of the scan program, it tries to find terraserver related objects;
 which are no longer in the default build.

Reactivate doc subtree

Previously, in order to build the reference documentation only when needed
(read: only some maintainers and only when they want) a solution was created
to deactivate processing of the whole doc subtree. But the side effect was to
not deliver at all the doc subtree if the correct options are not activated.
With an incomplete delivery, some tools like autoreconf (used by packaging
utilities) fail, bringing packager to patch and patch again the sources.

With this change, the doc subtree is no more ignored and the reference
documentation is processed ony when the corresponding ./configure flags is
activated.

Ensure thumbnails are created when opening JPG files.

Insert parameter for Viking's version in viking.xml

Insertion is done by configure, so the file is renamed as viking.xml.in.
The most annoying with this change is probably to configure the XML editor
to handle xml.in files.

Fix 2 heap-buffer-overflows in trw_layer_download_map_along_track_cb()

1 byte is less than sizeof(gpointer), so we do not have enough memory
for last NULL pointer.

This could be solved by braces, but since glib provides g_malloc_n(),
and this kind of bug can be avoided with it, so.

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Fix heap-buffer-overflow on Layers -> New GPS Layer

Very typical issue, unfortunately, missing braces and we are
corrupting memory. It is better to use g_malloc_n() to avoid this.

Fixes following AddressSanitizer error:
==17766==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x60b0008db6a0 at pc 0x00000048ba9e bp 0x7ffcf3e532a0 sp 0x7ffcf3e53298
WRITE of size 8 at 0x60b0008db6a0 thread T0
    0 0x48ba9d in gps_layer_inst_init /home/lebedevri/src/viking/src/vikgpslayer.c:386
    1 0x7fa95d549f98 in g_type_create_instance (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x31f98)
    2 0x7fa95d52d866  (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x15866)
    3 0x7fa95d52f25c in g_object_newv (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x1725c)
    4 0x7fa95d52f9f3 in g_object_new (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x179f3)
    5 0x48dca7 in vik_gps_layer_new /home/lebedevri/src/viking/src/vikgpslayer.c:659
    6 0x48e1f0 in vik_gps_layer_create /home/lebedevri/src/viking/src/vikgpslayer.c:419
    7 0x45d119 in vik_layer_create /home/lebedevri/src/viking/src/viklayer.c:229
    8 0x460463 in vik_layers_panel_new_layer /home/lebedevri/src/viking/src/viklayerspanel.c:461
    9 0x46982e in menu_addlayer_cb /home/lebedevri/src/viking/src/vikwindow.c:2411
    10 0x7fa95d5282d4 in g_closure_invoke (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x102d4)
    11 0x7fa95d53a03b  (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2203b)
    12 0x7fa95d542697 in g_signal_emit_valist (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2a697)
    13 0x7fa95d5428fe in g_signal_emit (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2a8fe)
    14 0x7fa95f0ec8cf  (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x748cf)
    15 0x7fa95d528503  (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x10503)
    16 0x7fa95d541fa6 in g_signal_emit_valist (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x29fa6)
    17 0x7fa95d5428fe in g_signal_emit (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2a8fe)
    18 0x7fa95f2c0c55 in gtk_widget_activate (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x248c55)
    19 0x7fa95f1bc98c in gtk_menu_shell_activate_item (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x14498c)
    20 0x7fa95f1bcd2a  (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x144d2a)
    21 0x7fa95f1aaa7e  (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x132a7e)
    22 0x7fa95d5282d4 in g_closure_invoke (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x102d4)
    23 0x7fa95d539f31  (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x21f31)
    24 0x7fa95d5421a4 in g_signal_emit_valist (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2a1a4)
    25 0x7fa95d5428fe in g_signal_emit (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2a8fe)
    26 0x7fa95f2c1ecb  (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x249ecb)
    27 0x7fa95f1a91c3 in gtk_propagate_event (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x1311c3)
    28 0x7fa95f1a965a in gtk_main_do_event (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x13165a)
    29 0x7fa95ee1abbb  (/usr/lib/x86_64-linux-gnu/libgdk-x11-2.0.so.0+0x5abbb)
    30 0x7fa95d251c3c in g_main_context_dispatch (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x49c3c)
    31 0x7fa95d251f1f  (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x49f1f)
    32 0x7fa95d252241 in g_main_loop_run (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x4a241)
    33 0x7fa95f1a85d6 in gtk_main (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x1305d6)
    34 0x41829c in main /home/lebedevri/src/viking/src/main.c:259
    35 0x7fa959a69b44 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x21b44)
    36 0x417b78  (/home/lebedevri/src/viking/src/viking+0x417b78)

0x60b0008db6a1 is located 0 bytes to the right of 97-byte region [0x60b0008db640,0x60b0008db6a1)
allocated by thread T0 here:
    0 0x7fa95f96437a in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x9437a)
    1 0x7fa95d257799 in g_malloc (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x4f799)
    2 0x7fa95d549f98 in g_type_create_instance (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x31f98)

SUMMARY: AddressSanitizer: heap-buffer-overflow /home/lebedevri/src/viking/src/vikgpslayer.c:386 gps_layer_inst_init
Shadow bytes around the buggy address:
..
  0x0c16801136c0: fa fa fa fa fa fa fa fa 00 00 00 00 00 00 00 00
=>0x0c16801136d0: 00 00 00 00[01]fa fa fa fa fa fa fa fa fa 00 00
..
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07
  Heap left redzone:       fa
==17766==ABORTING

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

SF Bugs#121: Additional race condition crash protection.

Subsequent code where variables could point to invalid locations.

Note this is called manually, so much less likely to encounter the race condition.

SF Bugs#121: Fix crash when invalidating previously acquired tiles.

This race condition can most often be triggered by zooming in/out rapidly.

The fault is due to using accessing variables outside of the lock section of the memory cache.
This has been in the code since the initial lock adding commit [in 2008]
 (SHA:9b79169dba7cd4df6badebdda23a370064ebfafd)

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

QA: extract effective code from g_assert

Typos spotted by coverity: https://scan.coverity.com/projects/viking