Re: wtls branch merged

"Nikos Balkanas" <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Message-ID <7266D4BBFEB54DB081EB8975E96B173F@drwho>
Hi,

Reporting from Solaris 10.5 amd64, 64bit compilation. 
Configured --with-wtls=openssl

1) Compilation: Clean. A couple of unrelated warnings fixed. Attaching 
patch.

2) Emulators used:

a) Openwave SDK 6.2.2 wap: no problems (connection tested)
b) Nokia NMBS 4.0: no problems (connection & connectionless tested)

Sites tested, following through links:

http://wap.google.com
http://wap.yahoo.com
http://m.facebook

Only facebook had a warning with nokia's emulator (b) about unsupported 
content. This was not observed with Openwave (a) and in any case it is 
related to wap, not wtls. The same happens in plain wtp communication.

Overall a succesful merge.

Thanks,
Nikos
----- Original Message ----- 
From: "Alexander Malysh" <[email protected]>
To: "Kannel Devel" <[email protected]>
Cc: "Nikos Balkanas" <[email protected]>
Sent: Sunday, September 12, 2010 1:04 PM
Subject: wtls branch merged


> Hi together,
>
> just merged and commited wtls branch into trunk.
> Please check it and let me know if something went wrong.
>
> Thanks,
> Alexander Malysh
kannel.diff (application/octet-stream, 999 B)
Index: wmlscript/wsgram.y
===================================================================
--- wmlscript/wsgram.y	(revision 4841)
+++ wmlscript/wsgram.y	(working copy)
@@ -720,7 +720,7 @@
 yyerror(char *msg)
 {
 #if WS_DEBUG
-  fprintf(stderr, "*** %s:%d: wsc: %s - this msg will be removed ***\n",
+  fprintf(stderr, "*** %s:%ld: wsc: %s - this msg will be removed ***\n",
 	  global_compiler->input_name, global_compiler->linenum, msg);
 #endif /* WS_DEBUG */
 }
Index: gwlib/pki.c
===================================================================
--- gwlib/pki.c	(revision 4841)
+++ gwlib/pki.c	(working copy)
@@ -93,7 +93,6 @@
 		
         /* Check errors!!!! */
         FILE* fp;
-        Octstr* foo;
         
         /* Open the file specified by "s" */
 		filename = octstr_get_cstr(s);
@@ -118,7 +117,6 @@
 		
         /* Check errors!!!! */
         FILE* fp;
-        Octstr* foo;
 
 		filename = octstr_get_cstr(s);
 		password = passwd != NULL ? octstr_get_cstr(passwd) : NULL;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.