Content

Bug #613

QML Launcher - Invalid pointer

Added by Herman van Hazendonk almost 5 years ago. Updated almost 5 years ago.

Status:Closed Start date:09/15/2014
Priority:Normal Due date:
Assignee:Nikolay Nizov % Done:

100% Total progress

Category:Applications
Target version:Black Eye

Description

Noticed this in the log, doesn't look good to me. Doesn't seem to have any influence from what I can tell, but still...

Sep 15 13:23:21 mako LunaAppManager1263: * Error in `/usr/sbin/luna-qml-launcher': munmap_chunk(): invalid pointer: 0x40f4e7c0 *


Issue Hierarchy ()

History

Updated by Simon Busch almost 5 years ago

  • Assignee deleted (Simon Busch)

Updated by Simon Busch almost 5 years ago

I saw this once you close a QML app.

Updated by Simon Busch almost 5 years ago

  • Target version changed from Antoccino to Black Eye

Updated by Nikolay Nizov almost 5 years ago

This is a bug in qtwayland. Issue was solved 2 days ago in the upstream by https://qt.gitorious.org/qt/qtwayland/commit/c7a8b70b4e9afef864652cf98a934d99d49bb36f
Seems https://qt.gitorious.org/qt/qtwayland/commit/4fdd9380a68fe0f9578ce9e307bef53cd91dc4a3 is also necessary to prevent other segfaults.
As those commits are in 5.4 branch, not sure how to fix bug. Maybe it is reasonable to manually push them for now, because bug is pretty important(crash on almost every close of app).

Updated by Herman van Hazendonk almost 5 years ago

I think manual patch for now would be good, since it will be a while still (probably another 2 releases or so ) before we'll be on 5.4 (still not released). Can you get a PR in for this and morphis can review?

Updated by Nikolay Nizov almost 5 years ago

OK, I will try to make PR, need to check how to grab commits from other authors and resolve commit dependencies.

Updated by Nikolay Nizov almost 5 years ago

Made https://github.com/webOS-ports/qtwayland/pull/1 ,but despite this bug is solved, appeared another with the same message(not appears in valgrind though).
That one is solved by https://github.com/webOS-ports/luna-next/pull/79

Updated by Christophe Chapuis almost 5 years ago

Thanks!
Regarding https://github.com/webOS-ports/luna-next/pull/79, shouldn't we also delete that pointer when LunaNext exits ?

Updated by Nikolay Nizov almost 5 years ago

Ah, forgot about this! But I think this is not necessary because it should be released by qqmlengine. Even if not, then it will be released on LunaNext termination. Those classes are pretty simple, without destructors, so I think it is OK.

Updated by Simon Busch almost 5 years ago

  • Assignee set to Nikolay Nizov
  • Status changed from New to In Progress

How far are we with this?

Updated by Simon Busch almost 5 years ago

  • % Done changed from 0 to 100
  • Status changed from In Progress to Closed

Merged all changes.

Also available in: Atom PDF