Creating Reservations Uses Memory after Free

Description

Found while running SmokeTest with AddressSanitizer.

In req_rescq.c::req_confirmresv, after logging the event, the server replies to the client saying their reservation is confirmed. In doing so, it calls reply_ack, which frees the batch_request. After replying, the server accesses the batch_request to write to the accounting log.

This was introduced in the fix of

Acceptance Criteria

None

Activity

Show:
Vincent Stumpf
November 30, 2017, 1:13 AM

Added logs.
Note: The failure of testing pbs_probe is due to a different bug, PP-1121.

Vincent Stumpf
December 6, 2017, 11:04 PM

Replaced logs for the new fix.

Assignee

Vincent Stumpf

Reporter

Vincent Stumpf

Severity

None

OS

None

Start Date

None

Pull Request URL

None

Story Points

1

Fix versions

Affects versions

Priority

Low
Configure