ChangeLog.git

   1 commit c560dc503a27d88e700ebb57be45da14c6b92fc6 (HEAD, refs/heads/master)
   2 Author: Michael Rash <mbr@cipherdyne.org>
   3 Date:   Tue Sep 4 21:47:30 2012 -0400
   4
   5     minor spelling typo fixes
   6
   7  ChangeLog |    8 ++++----
   8  1 file changed, 4 insertions(+), 4 deletions(-)
   9
  10 commit 25edd17aca0450844f89971ef1bcd3bb7aa231bc
  11 Author: Michael Rash <mbr@cipherdyne.org>
  12 Date:   Tue Sep 4 21:14:49 2012 -0400
  13
  14     diff update for fwknop-2.0.3 release
  15
  16  ChangeLog.git |  607 ++++++++++++++++++++-------------------------------------
  17  1 file changed, 216 insertions(+), 391 deletions(-)
  18
  19 commit 40ac28df21fab384f1389607eed78f6d35159206
  20 Author: Michael Rash <mbr@cipherdyne.org>
  21 Date:   Mon Sep 3 22:23:48 2012 -0400
  22
  23     bump version to 2.0.3
  24
  25  ChangeLog                    |    6 +++---
  26  VERSION                      |    2 +-
  27  android/project/jni/config.h |    6 +++---
  28  configure.ac                 |    2 +-
  29  fwknop.spec                  |    2 +-
  30  iphone/Classes/config.h      |    6 +++---
  31  lib/fko.h                    |    2 +-
  32  todo.org                     |    3 +++
  33  8 files changed, 16 insertions(+), 13 deletions(-)
  34
  35 commit 8d26cc90ee76ba95d58ee18d90431a9883a2a89a
  36 Author: Michael Rash <mbr@cipherdyne.org>
  37 Date:   Mon Sep 3 22:18:59 2012 -0400
  38
  39     include file compilation fix for OpenBSD relative to inet_aton() IP verification
  40
  41  lib/fko_message.c |    2 +-
  42  lib/fko_message.h |    9 +++++++--
  43  2 files changed, 8 insertions(+), 3 deletions(-)
  44
  45 commit b05d229bb15cb77a17a28a146b8b0dc61afa4aa9
  46 Author: Michael Rash <mbr@cipherdyne.org>
  47 Date:   Mon Sep 3 09:09:35 2012 -0400
  48
  49     sprintf() -> snprintf() calls
  50
  51  lib/fko_encryption.c |    6 ++++--
  52  1 file changed, 4 insertions(+), 2 deletions(-)
  53
  54 commit e2c0ac4821773eb335e36ad6cd35830b8d97c75a (refs/remotes/origin/master)
  55 Author: Michael Rash <mbr@cipherdyne.org>
  56 Date:   Mon Sep 3 00:21:46 2012 -0400
  57
  58     [server] Strong access.conf validation
  59
  60     Fernando Arnaboldi from IOActive found several conditions in
  61     which the server did not properly throw out maliciously constructed
  62     variables in the access.conf file.  This has been fixed along with new
  63     fuzzing tests in the test suite.
  64
  65  CREDITS                          |    6 ++
  66  ChangeLog                        |    4 ++
  67  Makefile.am                      |    3 +
  68  server/access.c                  |  114 +++++++++++++++++++++++++-------------
  69  test/conf/open_ports_access.conf |    4 +-
  70  test/test-fwknop.pl              |   42 ++++++++++++++
  71  6 files changed, 133 insertions(+), 40 deletions(-)
  72
  73 commit 263fa01f2af1d336961df320f1c7a9ea84ddac9a
  74 Author: Michael Rash <mbr@cipherdyne.org>
  75 Date:   Mon Sep 3 00:21:32 2012 -0400
  76
  77     added inet_aton() call for IP strong IP validation (credit: Fernando Arnaboldi)
  78
  79  lib/fko_message.c                            |   29 +++++++++++++++++++++-----
  80  lib/fko_message.h                            |    5 +++++
  81  test/conf/fuzzing_open_ports_access.conf     |    4 ++++
  82  test/conf/fuzzing_restrict_ports_access.conf |    5 +++++
  83  test/conf/fuzzing_source_access.conf         |    4 ++++
  84  5 files changed, 42 insertions(+), 5 deletions(-)
  85
  86 commit ffe4d3b162bbfea143704461aab4244cc4acdfcf
  87 Author: Michael Rash <mbr@cipherdyne.org>
  88 Date:   Sun Sep 2 15:53:54 2012 -0400
  89
  90     minor spacing update to make merges into hmac_master easier
  91
  92  test/test-fwknop.pl |   56 +++++++++++++++++++++++++--------------------------
  93  1 file changed, 28 insertions(+), 28 deletions(-)
  94
  95 commit 86b403dadb90c30deb51b3530e8ebbb791531615
  96 Author: Michael Rash <mbr@cipherdyne.org>
  97 Date:   Sat Sep 1 23:37:03 2012 -0400
  98
  99     fixed potential buffer overflow discovered by Fernando Arnaboldi of IOActive
 100
 101  server/access.c |   12 ++++++++++++
 102  1 file changed, 12 insertions(+)
 103
 104 commit e3a78a175c664ee51de1fb8086deb96a1d017ac3
 105 Author: Michael Rash <mbr@cipherdyne.org>
 106 Date:   Sat Sep 1 21:55:52 2012 -0400
 107
 108     verify_file_perms_ownership() to just return if the file doesn't exist
 109
 110  client/config_init.c |    2 +-
 111  client/fwknop.c      |    4 ++--
 112  client/utils.c       |   23 ++++++++++++++++-------
 113  server/fwknopd.c     |    3 ++-
 114  server/utils.c       |   25 +++++++++++++++++--------
 115  5 files changed, 38 insertions(+), 19 deletions(-)
 116
 117 commit 1548cbafc886af802b639913bb10e6a746222478
 118 Author: Michael Rash <mbr@cipherdyne.org>
 119 Date:   Fri Aug 31 23:05:05 2012 -0400
 120
 121     get MAX_PORT_STR_LEN constant from fko_message.h
 122
 123  client/spa_comm.c |    4 ++--
 124  client/utils.c    |    1 +
 125  common/common.h   |    2 --
 126  lib/fko.h         |    1 +
 127  4 files changed, 4 insertions(+), 4 deletions(-)
 128
 129 commit dafcfbc488f1e713ef6cfa9e86571a2b14e649d8
 130 Author: Michael Rash <mbr@cipherdyne.org>
 131 Date:   Fri Aug 31 23:00:45 2012 -0400
 132
 133     bug fix to make sure to verify file permissions/ownership on files that actually exist
 134
 135  client/fwknop.c |    6 ++----
 136  1 file changed, 2 insertions(+), 4 deletions(-)
 137
 138 commit b567514a6c722886fef5044a44abfc1514eff032
 139 Author: Michael Rash <mbr@cipherdyne.org>
 140 Date:   Fri Aug 31 22:59:44 2012 -0400
 141
 142     Added fko_context.h file to lib/Makefile.am
 143
 144  lib/Makefile.am |    3 ++-
 145  1 file changed, 2 insertions(+), 1 deletion(-)
 146
 147 commit d7393318189ace0a154823b359eb746aa0b36d94
 148 Author: Michael Rash <mbr@cipherdyne.org>
 149 Date:   Thu Aug 30 23:38:54 2012 -0400
 150
 151     implemented a couple of minor stronger bounds checks
 152
 153  client/config_init.c       |    5 +++++
 154  client/http_resolve_host.c |    6 +++---
 155  client/spa_comm.c          |    2 +-
 156  3 files changed, 9 insertions(+), 4 deletions(-)
 157
 158 commit 2584521c67952855ba20c7c61b701a34ba57615c
 159 Author: Michael Rash <mbr@cipherdyne.org>
 160 Date:   Thu Aug 30 21:43:53 2012 -0400
 161
 162     Run verify_file_perms_ownership() on fwknop.pid only if it exists
 163
 164     Two bugs are fixed with this commit: verify permissions/ownership on the
 165     fwknop.pid file only if it exists, and ensure to ru-run stat() on any directory
 166     component if we're creating a directory.
 167
 168  server/fwknopd.c |   20 ++++++++++++++++----
 169  1 file changed, 16 insertions(+), 4 deletions(-)
 170
 171 commit 406e33ccc0836796a53c88f7fe118d292adf0a25
 172 Author: Michael Rash <mbr@cipherdyne.org>
 173 Date:   Thu Aug 30 21:43:07 2012 -0400
 174
 175     minor comment update
 176
 177  server/utils.c |    4 ++--
 178  1 file changed, 2 insertions(+), 2 deletions(-)
 179
 180 commit 4832312e6de8401ac6fdbe63014ef7f186cf33cb
 181 Author: Michael Rash <mbr@cipherdyne.org>
 182 Date:   Wed Aug 29 23:12:56 2012 -0400
 183
 184     added filesystem permissions test
 185
 186  test/test-fwknop.pl |   41 +++++++++++++++++++++++++++++++++++++++--
 187  1 file changed, 39 insertions(+), 2 deletions(-)
 188
 189 commit a60f05ad44e824f6230b22f8976399340cb535dc
 190 Author: Michael Rash <mbr@cipherdyne.org>
 191 Date:   Wed Aug 29 22:21:43 2012 -0400
 192
 193     file permissions and client buffer overflow fix
 194
 195     - [client+server] Fernando Arnaboldi from IOActive found that strict
 196     filesystem permissions for various fwknop files are not verified.  Added
 197     warnings whenever permissions are not strict enough, and ensured that
 198     files created by the fwknop client and server are only set to user
 199     read/write.
 200     - [client] Fernando Arnaboldi from IOActive found a local buffer overflow
 201     in --last processing with a maliciously constructed ~/.fwknop.run file.
 202     This has been fixed with proper validation of .fwknop.run arguments.
 203
 204  ChangeLog             |    8 ++++++
 205  client/config_init.c  |   15 ++++++++---
 206  client/fwknop.c       |   19 ++++++++++---
 207  client/utils.c        |   66 +++++++++++++++++++++++++++++++++++++++++++--
 208  client/utils.h        |   13 +++++++++
 209  configure.ac          |    2 +-
 210  server/access.c       |    2 ++
 211  server/config_init.c  |    2 ++
 212  server/fwknopd.c      |    2 ++
 213  server/replay_cache.c |    6 ++++-
 214  server/utils.c        |   72 +++++++++++++++++++++++++++++++++++++++++++++++--
 215  server/utils.h        |    2 ++
 216  test/test-fwknop.pl   |   21 ++++++++++++++-
 217  13 files changed, 217 insertions(+), 13 deletions(-)
 218
 219 commit 186a424353a2e795e69f399f079a901e7dc8f24b
 220 Author: Michael Rash <mbr@cipherdyne.org>
 221 Date:   Tue Aug 28 21:28:57 2012 -0400
 222
 223     Added Ctrl-C and --disable-gpg notes
 224
 225  todo.org |    7 +++++++
 226  1 file changed, 7 insertions(+)
 227
 228 commit 098ae417fe91aefe501e9268aacd228374d0906d
 229 Author: Michael Rash <mbr@cipherdyne.org>
 230 Date:   Mon Aug 27 22:30:27 2012 -0400
 231
 232     migrated TODO tasks to the todo.org file
 233
 234  TODO     |   41 -----------------------------------------
 235  todo.org |   10 ++++++++++
 236  2 files changed, 10 insertions(+), 41 deletions(-)
 237
 238 commit 89dfa2c1fb06776646f99f722f21d47620f66695
 239 Author: Michael Rash <mbr@cipherdyne.org>
 240 Date:   Mon Aug 27 21:20:02 2012 -0400
 241
 242     minor ChangeLog update for the RPM build change
 243
 244  ChangeLog |    2 ++
 245  1 file changed, 2 insertions(+)
 246
 247 commit c5b229c5c87657197b0c814ff22127d870b55753
 248 Author: Michael Rash <mbr@cipherdyne.org>
 249 Date:   Mon Aug 27 21:16:59 2012 -0400
 250
 251     Added $DESTDIR prefix in uninstall-local and install-exec-hook to fix RPM builds
 252
 253  Makefile.am |   43 +++++++++++++++++++++----------------------
 254  1 file changed, 21 insertions(+), 22 deletions(-)
 255
 256 commit e8386dbe6c959365da5c08396e09c27901faed56
 257 Author: Michael Rash <mbr@cipherdyne.org>
 258 Date:   Sun Aug 26 15:47:24 2012 -0400
 259
 260     added encryption mode flags for each access stanza
 261
 262  server/access.c         |   17 ++++++++++++-----
 263  server/fwknopd_common.h |    3 +++
 264  server/incoming_spa.c   |   18 +++++-------------
 265  3 files changed, 20 insertions(+), 18 deletions(-)
 266
 267 commit 557cd6615b9cab21a9208390f5af070c66fd257d
 268 Author: Michael Rash <mbr@cipherdyne.org>
 269 Date:   Sun Aug 26 15:46:54 2012 -0400
 270
 271     consolidatd fuzzing functions within a single 'fuzzer' function
 272
 273  test/test-fwknop.pl |  449 +++++++++++++++++++--------------------------------
 274  1 file changed, 164 insertions(+), 285 deletions(-)
 275
 276 commit f4c16bc47fc24a96b63105556b62d61c1ba7d799
 277 Author: Michael Rash <mbr@cipherdyne.org>
 278 Date:   Sat Aug 25 23:08:55 2012 -0400
 279
 280     [server] Stronger IP validation based on a bug found by Fernando Arnaboldi from IOActive
 281
 282     This commit fixes a condition in which the server did not properly validate
 283     allow IP addresses from malicious authenticated clients.  This has been fixed
 284     with stronger allow IP validation.
 285
 286  CREDITS             |    2 ++
 287  ChangeLog           |    4 +++
 288  lib/fko_message.c   |   16 +++++++++---
 289  test/test-fwknop.pl |   67 +++++++++++++++++++++++++++++++++++++++++++++++++++
 290  4 files changed, 85 insertions(+), 4 deletions(-)
 291
 292 commit d46ba1c027a11e45821ba897a4928819bccc8f22
 293 Author: Michael Rash <mbr@cipherdyne.org>
 294 Date:   Fri Aug 24 22:12:19 2012 -0400
 295
 296     (Fernando Arnaboldi, IOActive) Found and fixed several DoS/code execution vulns for authenticated clients
 297
 298     - [server] Fernando Arnaboldi from IOActive found several DoS/code
 299     execution vulnerabilities for malicious fwknop clients that manage to
 300     get past the authentication stage (so a such a client must be in
 301     possession of a valid access.conf encryption key).  These vulnerbilities
 302     manifested themselves in the handling of malformed access requests, and
 303     both the fwknopd server code along with libfko now perform stronger input
 304     validation of access request data.  These vulnerabilities affect
 305     pre-2.0.3 fwknop releases.
 306     - [test suite] Added a new fuzzing capability to ensure proper server-side
 307     input validation.  Fuzzing data is constructed with modified fwknop
 308     client code that is designed to emulate malicious behavior.
 309
 310  CREDITS                              |    5 +
 311  ChangeLog                            |   13 ++
 312  Makefile.am                          |    1 +
 313  lib/fko_message.c                    |   23 +-
 314  lib/fko_message.h                    |    3 +
 315  server/access.c                      |   89 ++++++--
 316  server/access.h                      |    4 +-
 317  server/fw_util_iptables.c            |    3 +-
 318  test/conf/disable_aging_fwknopd.conf |    5 +
 319  test/test-fwknop.pl                  |  413 +++++++++++++++++++++++++++++++++-
 320  10 files changed, 531 insertions(+), 28 deletions(-)
 321
 322 commit b0bf7f369918989bae364730c8952258aac693c6
 323 Author: Michael Rash <mbr@cipherdyne.org>
 324 Date:   Sat Aug 18 16:30:34 2012 -0400
 325
 326     minor paren's syntax bug fix
 327
 328  server/incoming_spa.c |    2 ++
 329  1 file changed, 2 insertions(+)