Re: Two patches for better heimdal support

Hi Henrik,

That seems to be from another patch:

 bzr log -p -r11090 helpers/external_acl/kerberos_ldap_group/support_log.cc
------------------------------------------------------------
revno: 11090
committer: Amos Jeffries <squid3_at_treenet.co.nz>
branch nick: trunk
timestamp: Sat 2010-12-04 23:14:11 -0700
message:
  Fix invalid conversion 'long int' to 'time_t'
diff:
=== modified file 'helpers/external_acl/kerberos_ldap_group/support_log.cc'
--- helpers/external_acl/kerberos_ldap_group/support_log.cc 2010-08-14
00:12:49 +0000
+++ helpers/external_acl/kerberos_ldap_group/support_log.cc 2010-12-05
06:14:11 +0000
@@ -41,7 +41,8 @@

     gettimeofday(&now, NULL);
     if (now.tv_sec != last_t) {
- tm = localtime(&now.tv_sec);
+ time_t tmp = now.tv_sec;
+ tm = localtime(&tmp);
         strftime(buf, 127, "%Y/%m/%d %H:%M:%S", tm);
         last_t = now.tv_sec;
     }

"Henrik Nordström" <henrik_at_henriknordstrom.net> wrote in message
news:1291657930.10525.625.camel_at_henriknordstrom.net...
> The build farm now on kerberos_ldap_group due to int/time_t type
> mismatches
>
> ../../../../helpers/external_acl/kerberos_ldap_group/support_log.cc: In
> function `const char* LogTime()':
> ../../../../helpers/external_acl/kerberos_ldap_group/support_log.cc:44:
> error: invalid conversion from `long int*' to `const time_t*'
> ../../../../helpers/external_acl/kerberos_ldap_group/support_log.cc:44:
> error: initializing argument 1 of `tm* localtime(const time_t*)'
>
> I assume this is related to the heimdal support patches.
>
>
> Regards
> Henrik
>
>
Received on Mon Dec 06 2010 - 20:27:13 MST