Re: [389-devel] Please review: coverity 12606 Logically dead code

2012-03-15 Thread Mark Reynolds
ack On 03/15/2012 12:22 PM, Noriko Hosoi wrote: > The previous fix (commit 325abca7135d06225adf5380d726de60dacda5a4) > for "Ticket #303 - make DNA range requests work with transactions" > introduced this dead code. Since dna_pre_op does not allocate > an entry "e", there is no need to check the fl

[389-devel] Please review: coverity 12606 Logically dead code

2012-03-15 Thread Noriko Hosoi
The previous fix (commit 325abca7135d06225adf5380d726de60dacda5a4) for "Ticket #303 - make DNA range requests work with transactions" introduced this dead code. Since dna_pre_op does not allocate an entry "e", there is no need to check the flag "free_entry" and free it. >From 4e0c70fd1a52d4bc943613