Bug 6414 – Fix deadcode, leaks, and bad memory reads in GASS transfer

Bug 6414 - Fix deadcode, leaks, and bad memory reads in GASS transfer

Summary:

Fix deadcode, leaks, and bad memory reads in GASS transfer

Status:	RESOLVED FIXED

Product:	GridFTP
Component:	GASS
Version:	4.2.0
Platform:	All All

Importance:	P3 normal
Target Milestone:	4.2.1
Assigned To:	Joe Bester

URL:
Keywords:	4.0.x

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-09-24 11:44 by Joe Bester
Modified:	2008-09-25 13:49 (History)

Attachments
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 Joe Bester 2008-09-24 11:44:46

Deadcode:
    CID #79: constant condition in globus_l_gass_transfer_http_new_request()

Use after free:
    CID #2026: use proto after potential free in
globus_l_gass_transfer_http_read_callback()
    CID #2027: use proto after potential free in
globus_l_gass_transfer_http_write_callback()
    CID #2028: use proto after potential free in
globus_l_gass_transfer_http_writev_callback()
    CID #2029: use proto after potential free in
globus_l_gass_transfer_http_read_buffered_callback()

Leaks:
    CID #1228: leak in globus_l_gass_transfer_http_new_request()

------- Comment #1 From Joe Bester 2008-09-25 13:49:20 -------

Fixes committed to 4.2 branch, 4.0 branch, and trunk.