[asterisk-bugs] [Asterisk 0011813]: [patch] build error on main/tcptls.c when no ssl support is used

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11813 
====================================================================== 
Reported By:                tzafrir
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   11813
Category:                   General
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 99483 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             01-22-2008 04:45 CST
Last Modified:              01-23-2008 00:34 CST
====================================================================== 
Summary:                    [patch] build error on main/tcptls.c when no ssl
support is used
Description: 
When building trunk I get the following build error:

make[1]: Entering directory `/usr/src/asterisk-svn/main'
   [CC] tcptls.c -> tcptls.o
tcptls.c: In function ‘ast_make_file_from_fd’:
tcptls.c:359: error: ‘SSL_connect’ undeclared (first use in this
function)
tcptls.c:359: error: (Each undeclared identifier is reported only once
tcptls.c:359: error: for each function it appears in.)
tcptls.c:359: error: ‘SSL_accept’ undeclared (first use in this
function)
tcptls.c:361: warning: unused variable ‘err’
tcptls.c:360: warning: unused variable ‘ret’
tcptls.c:359: warning: unused variable ‘ssl_setup’
make[1]: *** [tcptls.o] Error 1
make[1]: Leaving directory `/usr/src/asterisk-svn/main'
make: *** [main] Error 2

$ svn info
Path: .
URL: http://svn.digium.com/svn/asterisk/trunk
Repository Root: http://svn.digium.com/svn/asterisk
Repository UUID: 614ede4d-c843-0410-af14-a771ab80d22e
Revision: 99483
Node Kind: directory
Schedule: normal
Last Changed Author: oej
Last Changed Rev: 99483
Last Changed Date: 2008-01-22 11:57:16 +0200 (Tue, 22 Jan 2008)

$ grep SSL include/asterisk/autoconfig.h
/* Define this to indicate the ${OPENSSL_DESCRIP} library */
/* #undef HAVE_OPENSSL */
/* Define to indicate the ${OPENSSL_DESCRIP} library version */
/* #undef HAVE_OPENSSL_VERSION */


There are two extra functions that have no "return" line in case DO_SSL is
not defined. The patch adds a return line with a silly value, which is why
it won't build as-is. This is because I was not sure what is a sane value
to return in case those functions are no-ops.
====================================================================== 

---------------------------------------------------------------------- 
 jamesgolovich - 01-23-08 00:34  
---------------------------------------------------------------------- 
This patch should take care of the issue of the lack of a return without
DO_SSL 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
01-23-08 00:34  jamesgolovich  Note Added: 0081057                          
======================================================================