[QPID-3205] NPE when setting the redelivery flag is delivery props are missing (or if there are no headers at all)

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.10
• Fix Version/s: 0.11
• Component/s: JMS AMQP 0-x
• Assignee: Rajith Muditha Attapattu
• Reporter: Rajith Muditha Attapattu
• Created: 12/Apr/11 16:56
• Updated: 14/Mar/17 20:08
• Resolved: 29/Jun/11 22:00

Description

In the 0-10 code path, when resuming a session we replay all the unacked messages in the replay buffer, after setting the redelivery flag.
However when doing so we need to check if the delivery properties are present.
The current implementation doesn't do so simply bcos the JMS client always sets delivery properties when sending messages.

However this results in a test failure (testResumeNonemptyReplayBuffer in Connection.java) and this code can be used in the future for QMF etc.. and they could potentially send messages without delivery props. So better to handle the null cases to avoid any NPE's.