Page MenuHomeSolus

T8298 - Fix geary consume all cpu due to upstream bug in handling some UTF in a mail header string
ClosedPublic

Authored by ArjenR on Thu, Aug 29, 7:45 AM.

Details

Summary

This patch resolves the issue mentioned in T8298 - Geary 3.32.1 high cpu due to upstream bug

Geary can continuously use all available cpu due to an upstream bug in handling some UTF in a mail header string
This is described here: https://gitlab.gnome.org/GNOME/geary/issues/504
A patch for the current mainline has been provided and was merged into mainline:
https://gitlab.gnome.org/GNOME/geary/commit/2025aa24e11219664ff4bd8f5017d63683b7dd45

I have cherry picked this commit into tags/geary-3.32.1 which applied without issue.

From there I created a patch file which I added to the solus geary build as a patch.
This builds cleanly and resolves the issue for me.

Test Plan

Upgrade an existing installation of Geary in which the issue can be observed and see if CPU consumption goes down after intial mailbox and mail folder scanning.

Diff Detail

Repository
R884 geary
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ArjenR created this revision.Thu, Aug 29, 7:45 AM
ArjenR requested review of this revision.Thu, Aug 29, 7:45 AM
davidjharder added a subscriber: davidjharder.EditedThu, Aug 29, 12:07 PM

Also, your diff should include the pspec_*.xml file and the ABI report: files starting with abi_*.

package.yml
3

Bump this to 24

Even if those were not changed? I added them to git and amended the commit. But since there are no changes in those, nothing extra get's picked up.

ArjenR updated this revision to Diff 16887.Thu, Aug 29, 12:21 PM

(tried to) add pspec_*.xml abi_*

ArjenR updated this revision to Diff 16888.Thu, Aug 29, 12:23 PM

bump release nr

chax added a subscriber: chax.Thu, Aug 29, 12:32 PM

You need to run make. That way you will get changes on those files. But first you need to setup local environment for solbuild.
Read articles that i linked in original issue. Start with this one: https://getsol.us/articles/packaging/building-a-package/en/

ArjenR updated this revision to Diff 16889.Thu, Aug 29, 1:01 PM
ArjenR marked an inline comment as done.

properly built for unstable

davidjharder added a comment.EditedThu, Aug 29, 2:39 PM

I should have checked this earlier: the geary repository has a MAINTAINERS file. This diff will probably be abandoned.

How about everybody except global maintainers and core team refrain from reviewing patches? Thanks.

JoshStrobl requested changes to this revision.Thu, Aug 29, 3:05 PM

Patch looks fine but you're required to use your real name for accountability purposes.

This revision now requires changes to proceed.Thu, Aug 29, 3:05 PM
ArjenR updated this revision to Diff 16901.Thu, Aug 29, 3:19 PM

Using real name for accountability purposes.

JoshStrobl accepted this revision.Thu, Aug 29, 3:44 PM

Thanks @ArjenR for the patch, LGTM!

This revision is now accepted and ready to land.Thu, Aug 29, 3:44 PM
This revision was automatically updated to reflect the committed changes.