On 03/11/2013 11:42 AM, Lukas Slebodnik wrote:
On (11/03/13 10:48), Ondrej Kos wrote:
> On 03/11/2013 10:22 AM, Lukas Slebodnik wrote:
>> diff --git a/src/util/util.h b/src/util/util.h
>> index
8afb3bc964dfc7ea5debf8127b7c6bac08d11f42..2adc8cb4180ae802afee63d084c3d1d67f273862 100644
>> --- a/src/util/util.h
>> +++ b/src/util/util.h
>> @@ -584,6 +584,9 @@ errno_t sssd_domain_init(TALLOC_CTX *mem_ctx,
>> /* from util_lock.c */
>> errno_t sss_br_lock_file(int fd, size_t start, size_t len,
>> int num_tries, useconds_t wait);
>> +/* from compact.c */
>
> if the file is renamed to io.c, please rewrite also this comment to
> avoid confusion
>
>> +int sss_open_cloexec(const char *pathname, int flags, int *ret);
>> +int sss_openat_cloexec(int dir_fd, const char *pathname, int flags, int *ret);
>
>
> --
> Ondrej Kos
> Associate Software Engineer
> Identity Management
> Red Hat Czech
>
> phone: +420-532-294-558
> cell: +420-736-417-909
> ext: 82-62558
> loc: 1013 Brno 1 office
> irc: okos @ #sssd #brno
Thank you for the notice, patches attached.
there are left unused variables after application of your patches,
remove these and I'll ack them
src/providers/ldap/sdap_child_helpers.c: In function 'setup_child':
src/providers/ldap/sdap_child_helpers.c:456:14: warning: unused variable
'v' [-Wunused-variable]
src/providers/krb5/krb5_init_shared.c: In function 'krb5_child_init':
src/providers/krb5/krb5_init_shared.c:33:14: warning: unused variable
'v' [-Wunused-variable]
--
Ondrej Kos
Associate Software Engineer
Identity Management
Red Hat Czech
phone: +420-532-294-558
cell: +420-736-417-909
ext: 82-62558
loc: 1013 Brno 1 office
irc: okos @ #sssd #brno