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 344132 - Variable "comp_str" is used before its value is set
Variable "comp_str" is used before its value is set
Status: RESOLVED FIXED
Product: evolution-data-server
Classification: Platform
Component: Calendar
1.8.x (obsolete)
Other All
: Normal normal
: ---
Assigned To: evolution-calendar-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2006-06-07 11:14 UTC by Marcin Garski
Modified: 2013-09-14 16:49 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16



Description Marcin Garski 2006-06-07 11:14:48 UTC
calendar/backends/groupwise/e-cal-backend-groupwise.c: (send_object):
There is a 'char *comp_str;' which is used in e_cal_backend_groupwise_modify_object() function call but it's value isn't set.
Comment 1 André Klapper 2006-08-28 12:43:19 UTC
i took a look into the CVS HEAD source file of the date you filed your bug, but i am unable to locate this here.
marcin, is this still an issue, or can you provide the line number or some surrounding code? thanks a lot in advance!
Comment 2 Marcin Garski 2006-08-28 15:48:54 UTC
You are right, you won't find it in CVS HEAD.
Please look at gnome-2-14 branch, 1.173.2.7 revision (newer one), send_object() function, case CAL_MODE_REMOTE : (line #2389).

Note: When I was currently looking at this file I noticed this:
In line nr 2377 you have a "char *uid, *comp_str;" but in line nr 2391 you shadow it.

I think that will help you.
Comment 3 Harish Krishnaswamy 2006-08-29 06:16:46 UTC
This method was not being called at all in the gnome-2-14 sources which explains why an ugly crasher was never spotted. The trunk has this implementation rewritten to support the retract feature. I have marked this as an unsupported method in that revision. 

Thanks for pointing that out :-).