Bug 626117 – crash in imapx_command_sync_changes_done

After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.

Bug 626117 - crash in imapx_command_sync_changes_done


Summary:	crash in imapx_command_sync_changes_done


Status:	RESOLVED DUPLICATE of bug 618845

Product:	evolution-data-server
Classification:	Platform
Component:	Mailer
Version:	2.30.x (obsolete)
Hardware:	Other Linux

Importance:	Normal critical
Target Milestone:	---
Assigned To:	evolution-mail-maintainers
QA Contact:	Evolution QA team

URL:
Whiteboard:	evolution[imapx]

Depends on:
Blocks:

Reported:	2010-08-05 15:26 UTC by Brian J. Murrell
Modified:	2010-10-01 12:44 UTC

See Also:
GNOME target:	---
GNOME version:	---

Description Brian J. Murrell 2010-08-05 15:26:06 UTC

Using gnome-2-30:

+ Trace 223119

Thread 1 (Thread 30999)

#0 imapx_command_sync_changes_done
at camel-imapx-server.c line 4069
#1 imapx_completion
at camel-imapx-server.c line 1811
#2 imapx_step
at camel-imapx-server.c line 1835
#3 parse_contents
at camel-imapx-server.c line 4205
#4 imapx_parser_thread
at camel-imapx-server.c line 4291
#5 g_thread_create_proxy
at /build/buildd/glib2.0-2.24.1/glib/gthread.c line 1893
#6 start_thread
at pthread_create.c line 300
#7 clone
at ../sysdeps/unix/sysv/linux/i386/clone.S line 130

Comment 1 Brian J. Murrell 2010-08-05 15:26:34 UTC

Bugzilla thinks this is a duplicate of bug 620023, FWIW.

Comment 2 Milan Crha 2010-09-10 06:35:41 UTC

Thanks for a bug report. It seems to me that the source code lines are somehow shifted, because my latest checkout of gnome-2-30 branch of evolution-data-server doesn't have line 4069 inside imapx_command_sync_changes_done. Is it possible you've there some patch applied, which shifts lines?

Nonetheless, there is not planned any update for 2.30 as of now, and the branch master contains quite many fixes in the whole imapx code, thus I would suggest to move to it.

Comment 3 Brian J. Murrell 2010-09-15 11:46:42 UTC

(In reply to comment #2)
> Thanks for a bug report. It seems to me that the source code lines are somehow
> shifted, because my latest checkout of gnome-2-30 branch of
> evolution-data-server doesn't have line 4069 inside
> imapx_command_sync_changes_done.

Hrm.  Strange.  Mine is the same, and it's the source that the crash should have come from.  Although, the report is now a month old, so my source might have been updated since then.

I currently have:

4065:	ic->pri = job->pri;
4066:	ic->job = job;
4067:	ic->complete = imapx_command_subscription_done;
4068:	imapx_command_queue(is, ic);
4069:
4070:	g_free (encoded_fname);
4071:}

> Is it possible you've there some patch
> applied, which shifts lines?

I don't think I have any imapx patches at the moment.

> Nonetheless, there is not planned any update for 2.30 as of now, and the branch
> master contains quite many fixes in the whole imapx code, thus I would suggest
> to move to it.

Is master stable enough yet?  Is it nearing branching for gnome-2-32?

Comment 4 Milan Crha 2010-09-15 12:24:00 UTC

Heh, bad line numbers. :)

There is a new branch gnome-2-32 created just recently, which is under hard code freeze till the upcoming 2.32.0 release. If you build yourself, then you probably can safely switch to it. The only trouble is a requirement for recent glib, but gtk can be 2.20.x with no issue.

Comment 5 Milan Crha 2010-10-01 12:44:02 UTC

Thanks for the bug report. This particular bug has already been reported into our bug tracking system, but please feel free to report any further bugs you find.

*** This bug has been marked as a duplicate of bug 618845 ***