<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Jan 10, 2015 at 1:40 AM, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Sat, Jan 10, 2015 at 01:35:59AM +0200, Miika Turkia wrote:<br>
> ><br>
> > new throws the crash, so that makes me think that there are more out of<br>
> > bounds errors here<br>
> > I’d suggest running this under Valgrind.<br>
><br>
> Good idea, but I probably won't have any time to do any hacking while here.<br>
> You know the drill. dive, eat, sleep, repeat<br>
<br>
</span>My last patch series fixed at least two bugs where we would write outside<br>
of the range of an array, either of which could corrupt the meta data and<br>
cause a crash like that.<span class=""><br></span></blockquote><div><br></div><div>No crashes on last download from the Vyper. I got a couple of error messages on the console:<br>[62.583698] ERROR: Failed to receive the answer. [in suunto_vyper2.c:207 (suunto_vyper2_device_packet)]<br><br></div><div>I suspect one during the initialization and once for each downloaded dive + one extra for good measure. But as far as I know, these were just error messages with no problems in the download.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
> > I fixed quite a few array access errors in the code, most likely I missed<br>
> > one or two.<br>
><br>
> I'll give it a try after I get 4 more dives on the DC.<br>
<br>
</span>:-)<br>
<span class=""><br>
> > I have an empty line on the divelog list, but one dive is missing. Thus<br>
> > I am able to download only 2 of todays 3 dives.<br>
<br>
</span>That one is fixed in the latest master.<span class=""><br></span></blockquote><div><br></div><div>Yep, this seems to be working now. All 4 from today downloaded successfully.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
> > It would be great if canceling download would allow me to select dives<br>
> > from the ones that were already loaded. (As I have to force load of all<br>
> > dives to get all 3 dives out of my Vyper, I naturally tried to speed things<br>
> > up and canceled the load prematurely.)<br>
<br>
</span>That is fixed in the latest master.<br>
<span class=""><br>
> > Are you able to build Subsurface while you are traveling?<br>
><br>
> Building myself is the only way to get new stuff under testing. Apt is not<br>
> working from here and I am too lazy to download the daily deb manually.<br>
<br>
</span>Makes sense. Latest master will make you much happier :-)</blockquote><div><br></div><div>indeed<br><br>miika <br></div></div></div></div>