[Date Prev][Date Next] [Chronological] [Thread] [Top]

RE: daemon.c unitialized value (ITS#1728)



This bug was fixed in the HEAD (daemon.c rev 1.204) last December but
apparently the patch hasn't made it back to the released code yet. An
oversight.

  -- Howard Chu
  Chief Architect, Symas Corp.       Director, Highland Sun
  http://www.symas.com               http://highlandsun.com/hyc
  Symas: Premier OpenSource Development and Support

> -----Original Message-----
> From: owner-openldap-bugs@OpenLDAP.org
> [mailto:owner-openldap-bugs@OpenLDAP.org]On Behalf Of
> Michael.Gerdts@alcatel.com
> Sent: Thursday, April 04, 2002 10:55 AM
> To: openldap-its@OpenLDAP.org
> Subject: daemon.c unitialized value (ITS#1728)
>
>
> Full_Name: Mike Gerdts
> Version: 2.0.23
> OS: Windows 2000
> URL: ftp://ftp.openldap.org/incoming/
> Submission from: (NULL) (143.209.230.142)
>
>
> When building OpenLDAP 2.0.23 from cvs (tag
> OPENLDAP_REL_ENG_2_0_23), a warning
> about an uninitialized variable used at line 542.  I believe that
> the warning is
> correct, as I can see no place that l.sl_addr is initialized.
>
>    535  #if defined( HAVE_GETADDRINFO ) && defined( HAVE_INET_NTOP )
>    536                  char addr[INET_ADDRSTRLEN];
>    537                  inet_ntop( AF_INET, &((struct sockaddr_in
> *)*sal)->sin_addr,
>    538                             addr, sizeof(addr) );
>    539                  s = addr;
>    540                  port = ((struct sockaddr_in *)*sal) ->sin_port;
>    541  #else
>    542                  s = inet_ntoa( l.sl_addr.sin_addr );
>    543                  port = l.sl_addr.sin_port;
>    544  #endif