On 01/29/2015 01:42 PM, Alexander Bokovoy wrote:
On Thu, 29 Jan 2015, Martin Babinsky wrote:
On 01/28/2015 06:30 PM, Alexander Bokovoy wrote:
On Wed, 28 Jan 2015, Martin Babinsky wrote:
On 01/28/2015 03:37 PM, Simo Sorce wrote:
On Wed, 2015-01-28 at 14:53 +0100, Martin Babinsky wrote:
On 01/27/2015 05:21 PM, Martin Babinsky wrote:
This series of patches is related to
https://fedorahosted.org/freeipa/ticket/4795.

The attached patches attempt to address some of the defects
encountered
during running covscan on freeipa-master branch.

The complete list is here:
http://cov01.lab.eng.brq.redhat.com/covscanhub/task/16553/log/freeipa-4.1.99.201501270952GITc90286c-0.fc21/scan-results.html




These patches do not fix defects 1 to 22 which occur in a code
generated
by asn1c, defect 23 which is related to
https://fedorahosted.org/freeipa/ticket/4861, and defect 27 (dead
code).

Each patch contains a detailed description of a defect and a proposed
fix to address it.

Martin^3


_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Attaching updated patches.


You are gratuitously removing empty lines in various patches.
They are there to make the code flow more readable, please do not remove
them.

Simo.

Attaching updated patches.

Patches look OK for me now. I need to test them in a clean install so
please don't commit them yet.


Some vim backup files crept into the previous attachment. Re-sending the
correct patchset.
ACK.


Good discussion&quick turnaround, thanks to all!

Pushed to:
master: 782ad366390e1672ebe3584b2c88f58d757e38b3
ipa-4-1: 5bad375656723595d60abba494992c27accebbe9

(links to all patches in the ticket)

Martin

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to