Latest changes

- Add support for Mac OS
- Add support for Raspberry Pi
- Fix BoringSSL compilation: include <openssl/hmac.h> explicitly

Author: Dmitri Tikhonov

CHANGELOG

@@ -0,0 +1,9 @@
+2017-09-26
+
+    - Add support for Mac OS
+    - Add support for Raspberry Pi
+    - Fix BoringSSL compilation: include <openssl/hmac.h> explicitly
+
+2017-09-22
+
+    - Initial release

CMakeLists.txt

@@ -28,7 +28,7 @@ MESSAGE(STATUS "DEVEL_MODE: ${DEVEL_MODE}")
 SET(MY_CMAKE_FLAGS "${MY_CMAKE_FLAGS} -Wall -Wextra -Wno-unused-parameter")
 SET(MY_CMAKE_FLAGS "${MY_CMAKE_FLAGS} -fno-omit-frame-pointer")
 
-IF(CMAKE_COMPILER_IS_GNUCC AND CMAKE_CXX_COMPILER_VERSION VERSION_LESS 4.9)
+IF(CMAKE_COMPILER_IS_GNUCC AND CMAKE_CXX_COMPILER_VERSION VERSION_LESS 4.9.3)
     SET(MY_CMAKE_FLAGS "${MY_CMAKE_FLAGS} -Wno-missing-field-initializers")
 ENDIF()
 IF(DEVEL_MODE EQUAL 1)
@@ -58,12 +58,15 @@ set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS}  ${MY_CMAKE_FLAGS} $ENV{EXTRA_CFLAGS}")
 
 MESSAGE(STATUS "Compiler flags: ${CMAKE_C_FLAGS}")
 
+IF(NOT DEFINED BORINGSSL_INCLUDE)
+    include_directories( /usr/local/lib )
+ENDIF()
+IF(NOT DEFINED BORINGSSL_LIB)
+    link_directories( /usr/local/include )
+ENDIF()
 
 SET(CMAKE_INCLUDE_CURRENT_DIR ON)
 include_directories( include )
-include_directories( ${BORINGSSL_INCLUDE} )
-
-link_directories( ${BORINGSSL_LIB} )
 
 IF(${CMAKE_SYSTEM_NAME} MATCHES "FreeBSD")
     # Find libevent on FreeBSD:
@@ -73,7 +76,7 @@ ENDIF()
 
 add_executable(http_client test/http_client.c test/prog.c test/test_common.c test/test_cert.c)
 
-target_link_libraries(http_client lsquic event pthread libssl.a libcrypto.a libdecrepit.a ${FIULIB} z m)
+target_link_libraries(http_client lsquic event pthread libssl.a libcrypto.a ${FIULIB} z m)
 
 add_subdirectory(src)
 

EXAMPLES.txt

@@ -11,6 +11,10 @@ Fetch Google's home page:
 
     ./http_client -H www.google.com -s 74.125.22.106:443 -p /
 
+    or even
+
+    ./http_client -H www.google.co.uk -s 2a00:1450:4009:80c::2003:443 -p /
+
 In the example above, -H specifies the domain; it is also used as the value
 of SNI paramater in the handshake.
 

README.md

@@ -21,17 +21,100 @@ Documentation
 
 The documentation for this module is admittedly sparse.  The API is
 documented in include/lsquic.h.  If you have doxygen, you can run
-`doxygen dox.cfg' or `make docs'.  The example program is
+`doxygen dox.cfg` or `make docs`.  The example program is
 test/http_client.c: a bare-bones, but working, QUIC client.  Have a look
 in EXAMPLES.txt to see how it can be used.
 
-Building
---------
+Requirements
+------------
 
-To build LSQUIC, you need CMake and BoringSSL.  The example program
-uses libevent to provide the event loop.  In short:
+To build LSQUIC, you need CMake, zlib, and BoringSSL.  The example program
+uses libevent to provide the event loop.
 
-  cmake -DBORINGSSL_INCLUDE=/some/dir -DBORINGSSL_LIB=/some/other/dir .
+Building BoringSSL
+------------------
+
+BoringSSL is not packaged; you have to build it yourself.  The process is
+straightforward.  You will need `go` installed.
+
+1. Clone BoringSSL by issuing the following command:
+
+```
+git clone https://boringssl.googlesource.com/boringssl
+cd boringssl
+```
+
+2. Check out stable branch:
+
+```
+git co chromium-stable
+```
+
+3. Compile the library
+
+```
+cmake . &&  make
+```
+
+If you want to turn on optimizations, do
+
+```
+cmake -DCMAKE_BUILD_TYPE=Release . && make
+```
+
+4. Install the library
+
+This is the manual step.  You will need to copy library files manually.
+LSQUIC client library needs two: `ssl/libssl.a` and `crypto/libcrypto.a`.
+To install these in `/usr/local/lib`, you should do the following:
+
+```
+BORINGSSL_SOURCE=$PWD
+cd /usr/local/lib
+sudo cp $BORINGSSL_SOURCE/ssl/libssl.a
+sudo cp $BORINGSSL_SOURCE/crypto/libcrypto.a
+```
+
+If you do not want to install the library (or do not have root), you
+can do this instead:
+
+```
+BORINGSSL_SOURCE=$PWD
+mkdir -p $HOME/tmp/boringssl-libs
+cd $HOME/tmp/boringssl-libs
+ln -s $BORINGSSL_SOURCE/ssl/libssl.a
+ln -s $BORINGSSL_SOURCE/crypto/libcrypto.a
+```
+
+Building LSQUIC Client Library
+------------------------------
+
+LSQUIC's `http_client` and the tests link BoringSSL libraries statically.
+Following previous section, you can build LSQUIC as follows:
+
+```
+cmake -DBORINGSSL_INCLUDE=$BORINGSSL_SOURCE/include \
+                                -DBORINGSSL_LIB=$HOME/tmp/boringssl-libs .
+make
+```
+
+Run tests:
+
+```
+make test
+```
+
+Platforms
+---------
+
+The client library has been tested on the following platforms:
+- Linux
+  - x86_64
+  - ARM (Raspberry Pi 3)
+- FreeBSD
+  - i386
+- MacOS
+  - x86_64
 
 Have fun,
 

src/liblsquic/lsquic_crypto.c

@@ -8,6 +8,7 @@
 #include <openssl/x509.h>
 #include <openssl/rand.h>
 #include <openssl/curve25519.h>
+#include <openssl/hmac.h>
 
 #include <zlib.h>
 

src/liblsquic/lsquic_full_conn.c

@@ -1907,12 +1907,6 @@ process_stream_ready_to_send (struct full_conn *conn, lsquic_stream_t *stream,
     {
         if (LSQUIC_STREAM_HANDSHAKE == stream->id)
         {
-#if LSQUIC_STREAM_HANDSHAKE
-            /* Full connection implies that the server has completed the
-             * handshake:
-             */
-            assert(0 == (conn->fc_flags & FC_SERVER));
-#endif
             /* Handshake messages are sent in brand-new packets.  If handshake
              * is not complete, the packet is zero-padded.
              */

src/liblsquic/lsquic_parse_gquic_be.h

@@ -13,6 +13,11 @@
 #define bswap_16 bswap16 
 #define bswap_32 bswap32 
 #define bswap_64 bswap64 
+#elif defined(__APPLE__)
+#include <libkern/OSByteOrder.h>
+#define bswap_16 OSSwapInt16
+#define bswap_32 OSSwapInt32
+#define bswap_64 OSSwapInt64
 #else
 #include <byteswap.h>
 #endif

src/liblsquic/lsquic_stream.c

@@ -498,13 +498,13 @@ lsquic_stream_write_avail (const lsquic_stream_t *stream)
         conn_avail = lsquic_conn_cap_avail(&stream->conn_pub->conn_cap);
         if (conn_avail < stream_avail)
         {
-            LSQ_DEBUG("stream %u write buffer is limited by connection: %jd",
-                stream->id, conn_avail);
+            LSQ_DEBUG("stream %u write buffer is limited by connection: "
+                "%"PRIu64, stream->id, conn_avail);
             return conn_avail;
         }
     }
 
-    LSQ_DEBUG("stream %u write buffer is limited by stream: %jd",
+    LSQ_DEBUG("stream %u write buffer is limited by stream: %"PRIu64,
         stream->id, stream_avail);
     return stream_avail;
 }

test/CMakeLists.txt

@@ -1,5 +1,22 @@
 # Copyright (c) 2017 LiteSpeed Technologies Inc.  See LICENSE.
 
+INCLUDE(CheckSymbolExists)
+
+CHECK_SYMBOL_EXISTS(
+    IP_MTU_DISCOVER
+    "netinet/in.h"
+    HAVE_IP_MTU_DISCOVER
+)
+
+CHECK_SYMBOL_EXISTS(
+    IP_DONTFRAG
+    "netinet/in.h"
+    HAVE_IP_DONTFRAG
+)
+
+
+CONFIGURE_FILE(${CMAKE_CURRENT_SOURCE_DIR}/test_config.h.in ${CMAKE_CURRENT_SOURCE_DIR}/test_config.h)
+
 
 add_subdirectory(unittests)
 

test/prog.c

@@ -15,6 +15,7 @@
 #include "../src/liblsquic/lsquic_hash.h"
 #include "../src/liblsquic/lsquic_logger.h"
 
+#include "test_config.h"
 #include "test_cert.h"
 #include "test_common.h"
 #include "prog.h"
@@ -86,7 +87,9 @@ prog_print_common_options (const struct prog *prog, FILE *out)
 "                 is used.\n"
 "   -i USEC     Library will `tick' every USEC microseconds.  The default\n"
 "                 is %u\n"
+#if LSQUIC_DONTFRAG_SUPPORTED
 "   -D          Set `do not fragment' flag on outgoing UDP packets\n"
+#endif
 "   -z BYTES    Maximum size of outgoing UDP packets.  The default is 1370\n"
 "                 bytes for IPv4 socket and 1350 bytes for IPv6 socket\n"
 "   -L LEVEL    Log level for all modules.  Possible values are `debug',\n"
@@ -142,6 +145,7 @@ prog_set_opt (struct prog *prog, int opt, const char *arg)
     case 'i':
         prog->prog_period_usec = atoi(arg);
         return 0;
+#if LSQUIC_DONTFRAG_SUPPORTED
     case 'D':
         {
             struct service_port *sport = TAILQ_LAST(prog->prog_sports, sport_head);
@@ -150,6 +154,7 @@ prog_set_opt (struct prog *prog, int opt, const char *arg)
             sport->sp_flags |= SPORT_DONT_FRAGMENT;
         }
         return 0;
+#endif
     case 'm':
         prog->prog_packout_max = atoi(arg);
         return 0;