Re: CVS commit: pkgsrc/security/zoneminder

[Joerg Sonnenberger <joerg%bec.de@localhost>]

On Fri, Jul 13, 2018 at 02:05:12AM +0000, Greg Troxel wrote:
> Module Name:  pkgsrc
> Committed By: gdt
> Date:         Fri Jul 13 02:05:12 UTC 2018
> 
> Modified Files:
>       pkgsrc/security/zoneminder: Makefile distinfo
>       pkgsrc/security/zoneminder/patches: patch-src_zm_logger_cpp
> Added Files:
>       pkgsrc/security/zoneminder/patches: patch-src_zm__event.cpp
>           patch-src_zm__rtp__source.cpp patch-src_zmu.cpp
> 
> Log Message:
> zoneminder: avoid %ld for time_t (int64_t, not long on arm)
> 
> zoneinder uses %ld to print tv.tv_sec, which is of type time_t.  On
> NetBSD, that's int64_t, which happens to match long on amd64, but not
> on arm, and hence printf often segfaults.  Kludge around this by
> casting to long, which should work for about 20 years, by which time a
> proper fix should have arrived in a zoneminder release.

The correct fix would be to use %jd and cast to intmax_t.

Joerg