Page MenuHomePhorge

Fix crashes when the XML parsing fails.
ClosedPublic

Authored by dfaure on Aug 1 2017, 12:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 3:28 PM
Unknown Object (File)
Wed, Apr 24, 7:04 AM
Unknown Object (File)
Tue, Apr 23, 1:51 PM
Unknown Object (File)
Tue, Apr 16, 3:46 AM
Unknown Object (File)
Tue, Apr 9, 7:20 AM
Unknown Object (File)
Thu, Mar 28, 12:57 AM
Unknown Object (File)
Mar 22 2024, 7:44 AM
Unknown Object (File)
Mar 11 2024, 10:48 AM
Subscribers
None

Details

Summary

This code was dereferencing a null pointer, leading to a repeatable
crash of the kolab resource when parsing an invalid event.

Diff Detail

Repository
rLK libkolab
Branch
libkolab-1.0
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 11477
Build 9239: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Aug 9 2017, 5:18 PM

Thanks for the review.

However I can't push, do you know why? My SSH key in git.kolab.org looks correct.

BUILDS PASSED Harbormaster builds for the active diff completed successfully.
PUSHING Pushing changes to "origin/libkolab-1.0".
Exception: You do not have permission to push to this repository.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

My git remote for pushing is ssh://git@git.kolab.org/diffusion/LK/libkolab.git/
Did something change?

In D503#5876, @dfaure wrote:

Thanks for the review.

However I can't push, do you know why? My SSH key in git.kolab.org looks correct.

BUILDS PASSED Harbormaster builds for the active diff completed successfully.
PUSHING Pushing changes to "origin/libkolab-1.0".
Exception: You do not have permission to push to this repository.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

My git remote for pushing is ssh://git@git.kolab.org/diffusion/LK/libkolab.git/
Did something change?

Because you removed yourself from the "Library Developers" group which is what gives you push access ;-) I have readded you to the group, hope that works now.

This revision was automatically updated to reflect the committed changes.

I really can't see why I would have removed myself from the lib dev group, nor can I remember doing that, but in any case, it works now. Thanks!