narrow down and fix applypatch access error #112

Closed
thestinger opened this Issue Dec 24, 2015 · 7 comments

Comments

Projects
None yet
3 participants
@thestinger
Contributor

thestinger commented Dec 24, 2015

This seems like it could be the same memory access error showing up elsewhere.

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 27, 2015

Contributor
Contributor

thestinger commented Dec 27, 2015

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 27, 2015

Contributor

This has now been narrowed down to being caused by the Bionic changes. Need to narrow it down to a specific change next. Hopefully it's the same issue causing the Chromium/WebView memory access errors.

Contributor

thestinger commented Dec 27, 2015

This has now been narrowed down to being caused by the Bionic changes. Need to narrow it down to a specific change next. Hopefully it's the same issue causing the Chromium/WebView memory access errors.

@thestinger thestinger self-assigned this Dec 27, 2015

@vanitasvitae

This comment has been minimized.

Show comment Hide comment
@vanitasvitae

vanitasvitae Dec 27, 2015

Nice 👍

Nice 👍

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 27, 2015

Contributor

This is now fixed. It wasn't the same issue as the browser issues. However, the pthread_atfork memory protection code in Bionic had to be extended to cover the newly added code path (in Marshmallow) for removing handlers and that is likely the cause of those issues. Both fixes will be included in the next release.

Contributor

thestinger commented Dec 27, 2015

This is now fixed. It wasn't the same issue as the browser issues. However, the pthread_atfork memory protection code in Bionic had to be extended to cover the newly added code path (in Marshmallow) for removing handlers and that is likely the cause of those issues. Both fixes will be included in the next release.

@thestinger thestinger closed this Dec 27, 2015

@polyzen

This comment has been minimized.

Show comment Hide comment
@polyzen

polyzen Dec 27, 2015

\o/

polyzen commented Dec 27, 2015

\o/

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 27, 2015

Contributor

A new release with the fixes is now available. Ignore the screwed up metadata on the downloads page. The over-the-air update server needs to be updated to cope with block-based updates.

Contributor

thestinger commented Dec 27, 2015

A new release with the fixes is now available. Ignore the screwed up metadata on the downloads page. The over-the-air update server needs to be updated to cope with block-based updates.

@vanitasvitae

This comment has been minimized.

Show comment Hide comment
@vanitasvitae

vanitasvitae Dec 27, 2015

Its working again! Thats great 👍 Thank you for the image from the past :D

Its working again! Thats great 👍 Thank you for the image from the past :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment