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

DOC: Fix typo

DOC: fixed typo

Fixed typo on the bzip2 option's name in configure

To fix this error, there were two solutions:
- change the name of the option
- change the description
I decided to change the name of the option.

[QA] Extra info on magic usage.

[QA] Extra info when using bzip2.

SF BugsZZ#123: Fix bzip2 decompression on Windows.

Need to specify opening as a binary file otherwise BZ2 read fails on Windows.

Also remember to cleanup resources - particularly closing the file.
Otherwise on Windows the subsequent remove + rename file operations fail as the file is still open.

Fix Geocaches acquiring with latest geo-* software

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

Mapnik3 support.

C++ standard required is now C++11.
New location for plugins.
Revised code for accessing Mapnik parameters and some types/functions have been renamed.

More correct menu icon

Don't attempt download if temporary file can't be opened.

[QA] Fix spelling in a comment and add function comment

Ensure layer visibility is retained when copying layers.

SF Bugs#121: Provisional fix

Protection against storing corrupt pixbufs into the mapcache may help prevent crashes seen in #121.

[WINDOWS] Fix reading zip files on Windows due to compiler alignment defaults changing.

Read zip files contents which are stored with no compression.

[QA] Protection from large malloc failures

Fix logic for comparing timestamps

[QA] Use correct type in sizeof function for memory allocation.

Don't retain GTK Dialogs in memory.

Not entirely sure why these were created and then shown/hidden as it doesn't save much time compared to simply creating them on demand.

Latest versions of GTK Dialog are more complicated and seemingly respawns a thread every few seconds (presumably to monitor if files have been created/deleted), irrespective whether the dialog is shown or not.

Thus now destroying it we avoid this unnecessary thread (re)creation overhead.

Re-fix SF Features #82. Simplify usage of restoring the last used directory.

[QA] Don't need copy this string.

[QA] Use map ids

SF Bugs#122: Fix memory cache confusion with multiple "On Disk OSM Tile Format" maps

Ensure use of the correct map id comparison so the string to enable differing cache lookups is set.

Prevent Gtk-Critical warnings after viewing the Track property dialog.

Support all values of GPS Mode fixType.

[QA] Fix typo in (an unused) macro definition.

keep alpha channel on mapnik rendering, scale alpha to alpha of mapnik layer

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

SF Bugs#120: Fix Track drawing bug across the 180th Meridian.

Red Hat Bugzilla – Bug 1210403: Only download and process one Bing attribution list.

If making more than one download request to get the Bing attributions,
 but then updating (and using) a single instance variable from multiple threads,
 results in undefined behaviour (i.e. could lead to crashes).

Thus ensure only one request at a time is made.

Simplify debug output of loaded Bing attributions.

Don't need to print out every stage of the formulation of a single attribute structure.

[QA] Make function names more unique to simplify use of debugger breakpoints.

Make simple GPSBabel filter options use the updated acquire framework options.

These filter options don't have to be shell commands anymore and so should work cross platform.

Rework Acquire framework process function options for maintainability and understanding.

Instead of overloaded parameters, use a structure with fields for the specific processing types and gpsbabel options.

Thus rename functions and parameter variables, update the usage thereof and simplify code as applicable.

Fix stability of some 32 bit builds.

Forcing 64bit types seems to improve reliability of these timestamp operations.

Fix tracks and routes not displayed when downloaded via the GPS Layer.

Need to calculate the bounding box of these, which is performed in the post read function.

Fix availability of the Google routing direction engine

Prevent crashing on direction requests when no engine is available.

[QA] Rename routing engine function as it gives URLs

Fix display of map tile source information when it has extended characters.

Since the dialog text uses Pango markup, need to escape the input string.

[WINDOWS] Remove usage of 'filter with' functions as these don't work in Windows.

Since these functions are bash shell commands they only work on Unix like systems.

[WINDOWS] Fix a missing \ in a potential gpsbabel location.

Update Help to wiki conversion method as the wiki is now in Markdown.

datasources.xml file should be packaged

Fix warning in a test program.

[DOC] Update copyright year

Fix potential broken builds.

Annoyingly g_str_to_ascii() is actually a recent addition to GLIB,
 despite the manual not stating which version it is available in
 (hence I thought it was always available).

Releasing Viking 1.6

Fix new test programs to work under 'make distcheck'

Fix renamed file

[DOC] Improve formal docbook compliance.

Add more files that have i18n.

Debian #746815: Fix loading image thumbnails when some Unicode chars in the pathname

[QA] White space fixes

Detect gdk_pixbuf_save failures in thumbnail generation

Debian #761966: Remove an incorrect accelerator in the German translation.

Document previous translation updates

Import Launchpad translation updates

[DOC] Interpolate Times

Prevent potential infinite speeds in interpolating time function.

Some points may end up with the same time so simply remove them.

Interpolate timestamps of a track between first and last trackpoint

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

Extend TrackWaypoint layer tooltip to include the total length of routes.

Fix mislabelled distance markers when using Nautical Miles.

Add the missing case for Nautical Miles.

[QA] Fix (an unused) function parameter type.

Enable a method to copy the time in the Trackpoint and Waypoint Edit dialogs.

Enable invoking Web Tools via right click menus on selected layers or points.

Extend Web Tools to accept a coordinate parameter,
 so that it can use that location in forming the URL.

[QA] Prevent a Gtk-Critical warning.

[QA] Update comment regarding GPSBabel version for Overpass API query result handling.

Improve layout of the Edit Trackpoint dialog.

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

Fix initialization so that preferences aren't accessed too early.

Use a two stage pass, since several initialization functions now read preferences as part of their process.

This is due to the first time a_preferences_get() is called it loads any preferences values from disk,
 but of course for preferences not registered yet it can't actually understand them.
Thus subsequent initial attempts to get unregistered preferences would not return the expected values.

Fix toolbar preference types

Fix changed Wikimedia Toolserver GeoHack URL

Add built in support for OSM Query requests.

[QA] Normalize OSM URL.

[QA] Improve tool_edit_trackpoint_click() comments

Add advanced setting to create TrackWaypoint layers without asking for details.

Fix enable selecting trackpoints/routepoints via Edit Trackpoint tool when the other kind is invisible.

SF Bugs#118: Fix route point selection via the Edit Trackpoint tool.

Allow re-sizing the width of the layers panel to smaller than the initial value.

SF Bugs#117: Confirm capability with GPSD_API_MAJOR_VERSION=6 (gpsd 3.12 / libgps.so.22)

[WINDOWS] Update to GPSBabel 1.5.2

[WINDOWS] Enable faster building by using all available CPUs.

Use the correct variable name!

Fixes crashing under Windows and probably more reliable sublayer ordering.

[QA] Force integer type in calls to abs() as x,y in GdkEventButton are doubles.