Re: [Freeipa-devel] [PATCHES] 101-107 Fixes for various Coverity issues

2013-02-28 Thread Martin Kosek
On 02/27/2013 01:55 PM, Martin Kosek wrote: > On 02/27/2013 01:39 PM, Martin Kosek wrote: >> On 02/27/2013 12:35 PM, Sumit Bose wrote: >>> Hi, >>> >>> the attached patches 102-107 fix issues found by Coverity which are >>> tracked by tickets #3422-#3427 and remove an unused variable >>> (patch 101)

Re: [Freeipa-devel] [PATCHES] 94-99 Read and use per-service PAC type

2013-02-28 Thread Sumit Bose
On Thu, Feb 28, 2013 at 08:44:35AM +0100, Martin Kosek wrote: > On 02/27/2013 06:48 PM, Sumit Bose wrote: > > On Wed, Feb 27, 2013 at 08:37:18AM -0500, Simo Sorce wrote: > >> On Wed, 2013-02-27 at 11:58 +0100, Sumit Bose wrote: > >>> On Mon, Feb 25, 2013 at 04:35:20PM +0100, Martin Kosek wrote: > >

Re: [Freeipa-devel] [PATCHES] 94-99 Read and use per-service PAC type

2013-02-28 Thread Martin Kosek
On 02/28/2013 12:42 PM, Sumit Bose wrote: > On Thu, Feb 28, 2013 at 08:44:35AM +0100, Martin Kosek wrote: >> On 02/27/2013 06:48 PM, Sumit Bose wrote: >>> On Wed, Feb 27, 2013 at 08:37:18AM -0500, Simo Sorce wrote: On Wed, 2013-02-27 at 11:58 +0100, Sumit Bose wrote: > On Mon, Feb 25, 2013

[Freeipa-devel] [PATCH] 375 Fix includedir directive in krb5.conf template

2013-02-28 Thread Martin Kosek
We did not have the includedir directory with a trailing slash which made rpm update add a redundant line. https://fedorahosted.org/freeipa/ticket/3132 --- Pushed as a one-liner (with Rob's IRC blessing) to master, ipa-3-1, ipa-3-0. Martin From 68ac8aa9d999d7547eaf088a89f922dfb3bfd68d Mon Sep 1

Re: [Freeipa-devel] [PATCHES] 94-99 Read and use per-service PAC type

2013-02-28 Thread Simo Sorce
On Thu, 2013-02-28 at 13:02 +0100, Martin Kosek wrote: > On 02/28/2013 12:42 PM, Sumit Bose wrote: > > On Thu, Feb 28, 2013 at 08:44:35AM +0100, Martin Kosek wrote: > >> On 02/27/2013 06:48 PM, Sumit Bose wrote: > >> Hi Sumit, > >> > >> This looks like a good idea and would prevent the magic defau

Re: [Freeipa-devel] [PATCHES] 94-99 Read and use per-service PAC type

2013-02-28 Thread Martin Kosek
On 02/28/2013 03:28 PM, Simo Sorce wrote: > On Thu, 2013-02-28 at 13:02 +0100, Martin Kosek wrote: >> On 02/28/2013 12:42 PM, Sumit Bose wrote: >>> On Thu, Feb 28, 2013 at 08:44:35AM +0100, Martin Kosek wrote: On 02/27/2013 06:48 PM, Sumit Bose wrote: > Hi Sumit, This looks lik