<feed xmlns='http://www.w3.org/2005/Atom'>
<title>subsurface.git/smtk-import, branch v4.9.8</title>
<subtitle>forked from https://github.com/subsurface/subsurface</subtitle>
<id>https://git.tsegers.com/subsurface.git/atom?h=v4.9.8</id>
<link rel='self' href='https://git.tsegers.com/subsurface.git/atom?h=v4.9.8'/>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/'/>
<updated>2020-11-01T22:52:34Z</updated>
<entry>
<title>smtk-import: Add support for new devices model</title>
<updated>2020-11-01T22:52:34Z</updated>
<author>
<name>Salvador Cuñat</name>
<email>salvador.cunat@gmail.com</email>
</author>
<published>2020-11-01T21:32:43Z</published>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/commit/?id=1ff488a4392f4d9a41cf1652c351e7b59bc03928'/>
<id>urn:sha1:1ff488a4392f4d9a41cf1652c351e7b59bc03928</id>
<content type='text'>
Include devices Ids in the DC data.
Ensure we always set a DC model string for manual dives or unsupported devices.

Signed-off-by: Salvador Cuñat &lt;salvador.cunat@gmail.com&gt;
</content>
</entry>
<entry>
<title>cleanup: break out event-related code into event.[c|h]</title>
<updated>2020-10-25T20:59:52Z</updated>
<author>
<name>Berthold Stoeger</name>
<email>bstoeger@mail.tuwien.ac.at</email>
</author>
<published>2020-10-25T08:14:16Z</published>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/commit/?id=8212acc9925b28ecd546b01047c6a8fc574326ef'/>
<id>urn:sha1:8212acc9925b28ecd546b01047c6a8fc574326ef</id>
<content type='text'>
In an effort to reduce the size of dive.h and dive.c, break out
the event related functions. Moreover event-names were handled
by the profile-code, collect that also in the new source files.

Signed-off-by: Berthold Stoeger &lt;bstoeger@mail.tuwien.ac.at&gt;
</content>
</entry>
<entry>
<title>build-system: switch to using C++17 as default C++ dialect</title>
<updated>2020-10-24T22:21:55Z</updated>
<author>
<name>Berthold Stoeger</name>
<email>bstoeger@mail.tuwien.ac.at</email>
</author>
<published>2020-10-17T22:08:38Z</published>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/commit/?id=1211520ca939e53d6880e213dd38053a08b9589e'/>
<id>urn:sha1:1211520ca939e53d6880e213dd38053a08b9589e</id>
<content type='text'>
Signed-off-by: Berthold Stoeger &lt;bstoeger@mail.tuwien.ac.at&gt;
Signed-off-by: Dirk Hohndel &lt;dirk@hohndel.org&gt;
</content>
</entry>
<entry>
<title>smtk-import: Workaround segfault in mdbtools memcpy call</title>
<updated>2020-08-13T17:02:12Z</updated>
<author>
<name>Salvador Cuñat</name>
<email>salvador.cunat@gmail.com</email>
</author>
<published>2020-08-02T08:39:39Z</published>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/commit/?id=e3a158624bd5b15451626907a30185ec4aaf0d87'/>
<id>urn:sha1:e3a158624bd5b15451626907a30185ec4aaf0d87</id>
<content type='text'>
Smtk2ssrf has a segfault which matches quite well glibc's
CVE-2019-6488 (except for the x32 part).
It came from a call to memcpy in mdb_ole_read() func, used to get the
header and the profile of a dive from the database.
May be it could be fixed in libmdb but Mdbtools project has been stalled
for the past 5 years so ...

The segfault seems to be triggered by an empty profile in the first dive
in the database (a pretty common case in older Aladin DCs due to their
little memmory). The only special thing here is the fact it's the first
dive in the database structure (not the first by its index).

We can avoid the crash if we don't call mdb_ole_read_full() func on zero
sized profile field.

The problem here is we can't get the size of the fields and build the
MdbColumn in the same roud.  Happily we just need the MdbColumn struct
for the dive profile and header.  So, we can change the previous approach
using MdbColumns through almost all functions to a simpler one using the
already bounded strings by smtk_open_table() and just using the
col[n]-&gt;bind_pointer in the main function where the columns are built to
be used by mdb_ole_read_full().

Reported-by: Robert C. Helling &lt;helling@atdotde.de&gt;
Signed-off-by: Salvador Cuñat &lt;salvador.cunat@gmail.com&gt;
</content>
</entry>
<entry>
<title>build-system/MXE: build with more up to date MXE tools.</title>
<updated>2020-06-25T18:00:51Z</updated>
<author>
<name>Paul Buxton</name>
<email>paulbuxton.mail@googlemail.com</email>
</author>
<published>2020-05-30T11:21:22Z</published>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/commit/?id=a753845d5acd6d25ff74dc969a9ea6307d8abe6a'/>
<id>urn:sha1:a753845d5acd6d25ff74dc969a9ea6307d8abe6a</id>
<content type='text'>
- use hidapi grantlee and mdbtools from MXE
- update MXE version to use QT 5.15, and pull in libzstd and  CMake 3.17.3
- fix linking of winmm on windows build with new mxe
- add some instructions on building the container
- add some new dependancies from QT 5.15 to the packaging
- add a patch to MXE to Build qtconnectivity with native-win32-bluetooth

[Dirk Hohndel: small refactor]

Signed-off-by: Paul Buxton &lt;paulbuxton.mail@googlemail.com&gt;
Signed-off-by: Dirk Hohndel &lt;dirk@hohndel.org&gt;
</content>
</entry>
<entry>
<title>cleanup: move dive_table from dive.h to divelist.h</title>
<updated>2020-05-01T16:42:31Z</updated>
<author>
<name>Berthold Stoeger</name>
<email>bstoeger@mail.tuwien.ac.at</email>
</author>
<published>2020-05-01T11:43:52Z</published>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/commit/?id=95284c026e763482ef581c8b6298012d2c1fd941'/>
<id>urn:sha1:95284c026e763482ef581c8b6298012d2c1fd941</id>
<content type='text'>
This allows us to decouple dive.h and divelist.h, a small step in
include disentangling.

Signed-off-by: Berthold Stoeger &lt;bstoeger@mail.tuwien.ac.at&gt;
</content>
</entry>
<entry>
<title>[smtk-import] Fix useless NULL check and white space issues</title>
<updated>2020-04-19T17:59:12Z</updated>
<author>
<name>Salvador Cuñat</name>
<email>salvador.cunat@gmail.com</email>
</author>
<published>2020-04-19T16:44:24Z</published>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/commit/?id=5a0f2fc2ccadd97e5394f2f7ac879a191e44771b'/>
<id>urn:sha1:5a0f2fc2ccadd97e5394f2f7ac879a191e44771b</id>
<content type='text'>
As commented by Berthold and Dirk for previous patches.

Signed-off-by: Salvador Cuñat &lt;salvador.cunat@gmail.com&gt;
</content>
</entry>
<entry>
<title>[smtk-import] add some sanity checks</title>
<updated>2020-04-19T17:59:12Z</updated>
<author>
<name>Salvador Cuñat</name>
<email>salvador.cunat@gmail.com</email>
</author>
<published>2020-04-19T05:54:35Z</published>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/commit/?id=e139b7b15fbffba6cf302ec49a67239663144a5b'/>
<id>urn:sha1:e139b7b15fbffba6cf302ec49a67239663144a5b</id>
<content type='text'>
Under some circustances values stored in this arrays may be NULL or even
previously freed.  Check them to avoid further crashes.

Signed-off-by: Salvador Cuñat &lt;salvador.cunat@gmail.com&gt;
</content>
</entry>
<entry>
<title>[smtk-import] Do not create cylinders structure before libdc parsing</title>
<updated>2020-04-19T17:59:12Z</updated>
<author>
<name>Salvador Cuñat</name>
<email>salvador.cunat@gmail.com</email>
</author>
<published>2020-04-19T05:38:19Z</published>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/commit/?id=6c9a9b2fb5ff1d8e3aab9b041617349917ac5061'/>
<id>urn:sha1:6c9a9b2fb5ff1d8e3aab9b041617349917ac5061</id>
<content type='text'>
The d-&gt;cylinders created will be overriden by libdivecomputer parsing,
resulting in 0, 1 , may be 2 cylinders depending on DC data. This is not
what we want when importing a divelog, because we will miss all hand
entered tanks.
BTW, using get_cylinder() on tank number bigger than created, results in
a NULL pointer and a crash.
As we can't foresee how many tanks (or even it's positional numbers in
log) a diver has used, the full 10 tanks supported by SmarTrak can be
easily created and parsed using get_or_create_cylinder(), and unused
cleaned later.

Signed-off-by: Salvador Cuñat &lt;salvador.cunat@gmail.com&gt;
</content>
</entry>
<entry>
<title>Grammar: replaces 'indexes' by 'indices'</title>
<updated>2020-03-11T15:26:30Z</updated>
<author>
<name>Robert C. Helling</name>
<email>helling@atdotde.de</email>
</author>
<published>2020-03-11T10:30:51Z</published>
<link rel='alternate' type='text/html' href='https://git.tsegers.com/subsurface.git/commit/?id=285fa8acbceebf5dc87c5c3befe42a2917a3b844'/>
<id>urn:sha1:285fa8acbceebf5dc87c5c3befe42a2917a3b844</id>
<content type='text'>
Grammar-nazi ran

git grep -l 'indexes' | xargs sed -i '' -e 's/indexes/indices/g'

to prevent future wincing when reading the source code.

Unfortunatly, Qt itself is infected as in
QModelIndexList QItemSelection::indexes() const

Signed-off-by: Robert C. Helling &lt;helling@atdotde.de&gt;
</content>
</entry>
</feed>
