Bug 2035 – globus_callout_read_config sets datum->next to datum

Bug 2035 - globus_callout_read_config sets datum->next to datum

Summary:

globus_callout_read_config sets datum->next to datum

Status:	RESOLVED FIXED

Product:	GSI C
Component:	Authentication
Version:	3.9.2
Platform:	PC Linux

Importance:	P3 normal
Target Milestone:	---
Assigned To:	Sam Meder

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2004-10-07 10:09 by Tom Fredian
Modified:	2008-08-11 15:21 (History)

Attachments
Fix (1.64 KB, patch) 2004-10-09 21:37, Sam Meder	Details
Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description From Tom Fredian 2004-10-07 10:09:54

In the routine, globus_callout_read_config, line 372 of globus_callout.c does 
the following:

      datum->next = datum;

In the routine, globus_callout_call_type, around line 721 of globus_callout.c 
current_datum is set to current_datum->next and the do while(current_datum) 
loop goes into an infinite loop.

I discovered this when trying to configure an authz callout. I'm not sure why 
datum->next is set to itself instead of null at line 372 but this seems to be 
a bug.

Thanks,
Tom Fredian
Plasma Science and Fusion Center, MIT

------- Comment #1 From Sam Meder 2004-10-09 21:37:36 -------

Created an attachment (id=441) [details]
Fix

------- Comment #2 From Sam Meder 2004-10-09 21:38:17 -------

I've committed the attached fix to CVS. Thanks for the report.