-
Notifications
You must be signed in to change notification settings - Fork 0
/
ChangeLog
1931 lines (1267 loc) · 66.3 KB
/
ChangeLog
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
796
797
798
799
800
801
802
803
804
805
806
807
808
809
810
811
812
813
814
815
816
817
818
819
820
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
839
840
841
842
843
844
845
846
847
848
849
850
851
852
853
854
855
856
857
858
859
860
861
862
863
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
887
888
889
890
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931
932
933
934
935
936
937
938
939
940
941
942
943
944
945
946
947
948
949
950
951
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
969
970
971
972
973
974
975
976
977
978
979
980
981
982
983
984
985
986
987
988
989
990
991
992
993
994
995
996
997
998
999
1000
2013-07-12 Timo Teras <timo.teras@iki.fi>
* src/racoon/main.c: From Sven Vermeulen
<sven.vermeulen@siphos.be>: Moves ploginit() up, allowing logging
events from init_avc() to show up as well.
2013-06-18 Timo Teras <timo.teras@iki.fi>
* src/racoon/ipsec_doi.c: From Paul Barker: Remove redundant memset
after calloc that caused compile failures with gcc 4.8 due to error:
argument to 'sizeof' in 'memset' call is the same expression as the
destination; did you mean to dereference.
2013-06-03 Timo Teras <timo.teras@iki.fi>
* src/racoon/admin.c: From Alexander Sbitnev
<alexander.sbitnev@gmail.com>: fix admin port establish-sa for
tunnel mode SAs.
2013-05-23 Timo Teras <timo.teras@iki.fi>
* src/include-glibc/net/pfkeyv2.h: From Rainer Weikusat
<rweikusat@mobileactivedefense.com>: Fix SADB_X_EALG_CASTCBC
definition to use system definition (which differs at least on
Linux).
2013-04-12 Timo Teras <timo.teras@iki.fi>
* src/racoon/isakmp_cfg.c: From Rainer Weikusat
<rweikusat@mobileactivedefense.com>: Do not send out illegal zero
length MODE_CFG attributes.
* src/racoon/: grabmyaddr.c, isakmp_inf.c: Some logging
improvements.
2013-02-05 Timo Teras <timo.teras@iki.fi>
* src/racoon/grabmyaddr.c: Fix source port selection
* src/racoon/isakmp_xauth.c: From Ian West <ian@niw.com.au>: Fix
double free of the radius info on config reload.
2013-01-24 Timo Teras <timo.teras@iki.fi>
* src/racoon/isakmp_inf.c: Fix handling of deletion notification.
2013-01-08 tag ipsec-tools-0_8_1
2013-01-08 Timo Teras <timo.teras@iki.fi>
* NEWS, configure.ac: ipsec-tools-0.8.1
* configure.ac: Fix errors from automake 1.13
* src/include-glibc/Makefile.am: Don't derefence the directory
symlink which we might be recreating.
2012-12-24 Timo Teras <timo.teras@iki.fi>
* src/racoon/crypto_openssl.c: From Götz Babin-Ebell
<g.babin-ebell@novamedia.de>: Smarter X.509 subject name compare.
* configure.ac, src/racoon/crypto_openssl.c,
src/racoon/missing/crypto/sha2/sha2.c: From Götz Babin-Ebell
<g.babin-ebell@novamedia.de>: Require OpenSSL 0.9.8s or higher
2012-08-29 Timo Teras <timo.teras@iki.fi>
* src/racoon/isakmp_inf.c: From Roman Hoog Antink <rha@open.ch>:
Accept DPD messages with cookies also in reversed order for
compatiblity. At least Cisco 836 running IOS 12.3(8)T does this.
* src/racoon/oakley.c: From Roman Hoog Antink <rha@open.ch>: add
remote's IP address to the "certificate not verified" error message.
* src/racoon/oakley.c: From Roman Hoog Antink <rha@open.ch>: do not
print unnecessary warning about non-verified certificate when using
raw plain-rsa.
* src/racoon/isakmp.c: From Rainer Weikusat
<rweikusat@mobileactivedefense.com>: Release unused phase2 of
passive remotes after acquire.
* src/racoon/isakmp.c: From Wolfgang Schmieder
<wolfgang.schmieder@honeywell.com>: setup phase1 port properly.
* src/racoon/: cfparse.y, cftoken.l, racoon.conf.5: Allow inherited
remote blocks without additional remote statements to be specified
in a simpler way. patch by Roman Hoog Antink <rha@open.ch>
2012-08-23 Timo Teras <timo.teras@iki.fi>
* src/racoon/crypto_openssl.c: From Nakano Takaharu: Fix bignum
memory allocation.
2012-01-01 Timo Teras <timo.teras@iki.fi>
* src/racoon/isakmp_unity.c: From Rainer Weikusat
<rweikusat@mobileactivedefense.com>: Fix one byte too short memory
allocation in isakmp_unity.c:splitnet_list_2str().
2011-11-17 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/handler.c: fixed some crashes in LIST_FOREACH where
current element could be removed during the loop
2011-11-14 Timo Teras <timo.teras@iki.fi>
* src/libipsec/pfkey.c: From Marcelo Leitner <mleitner@redhat.com>:
do not shrink pfkey socket buffers (if system default is larger than
what we want as minimum)
2011-08-12 Timo Teras <timo.teras@iki.fi>
* src/racoon/privsep.c: Have privilege separation child process
exit if the parent exits.
* Makefile.am: Create ChangeLog for proper CVS branch.
2011-03-18 tag ipsec-tools-0_8_0
2011-03-18 Yvan Vanhullebus <vanhu@netasq.com>
* configure.ac: Yes: 0.8.0 is out !!!
* NEWS: updated News for 0.8 branch
2011-03-17 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/oakley.c: fixed a memory leak in
oakley_append_rmconf_cr() while generating plist. patch by Roman
Hoog Antink <rha@open.ch>
* src/racoon/oakley.c: free name later, to avoid a memory use after
free in oakley_check_certid(). also give iph1->remote to some plog()
calls. patch by Roman Hoog Antink <rha@open.ch>
* src/racoon/oakley.c: fixed a memory leak in
oakley_check_certid(). patch by Roman Hoog Antink <rha@open.ch>
2011-03-15 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/: isakmp.c, isakmp_inf.c, pfkey.c: directly call
isakmp_ph1delete() instead of scheduling isakmp_ph1delete_stub(), as
it is useless an can lead to memory access after free
2011-03-14 Timo Teras <timo.teras@iki.fi>
* src/racoon/: grabmyaddr.c, handler.c, isakmp.c, isakmp_inf.c,
isakmp_quick.c, nattraversal.c, pfkey.c, policy.c, sockmisc.c,
sockmisc.h, throttle.c: Explicitly compare return value of
cmpsaddr() against a return value define to make it more obvious
what is the intended action. One more return value is also added, to
fix comparison of security policy descriptors. Namely, getsp()
should not allow wildcard matching (as the comment says, it does
exact matching) - otherwise we get problems when kernel has generic
policy with no ports, and a second similar policy with ports.
2011-03-14 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/: cfparse.y, isakmp_xauth.c, isakmp_xauth.h,
remoteconf.c, remoteconf.h, rsalist.c, rsalist.h: avoid some
memory leaks / free memory access when reloading conf and have
inherited config. patch from Roman Hoog Antink <rha@open.ch>
* src/racoon/handler.c: removed an useless comment
* src/racoon/handler.c: check if we got RMCONF_ERR_MULTIPLE from
getrmconf_by_ph1() in revalidate_ph1tree_rmconf()
2011-03-11 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/: handler.c, isakmp.c: directly delete a ph1 in
remove_ph1-) instead of scheduling it, to avoid (completely ?) a
race condition when reloading configuration
2011-03-06 Timo Teras <timo.teras@iki.fi>
* src/racoon/privsep.c: Quiet a gcc warning when strict-aliasing
checks are enabled. Reported by Stephen Clark.
2011-03-02 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/session.c: flush sainfo list when closing session.
patch by Roman Hoog Antink <rha@open.ch>
* src/racoon/: remoteconf.c, rsalist.c, rsalist.h: free rsa
structures when deleting a struct rmconf. patch by Roman Hoog Antink
<rha@open.ch>
* src/racoon/: cfparse.y, remoteconf.c, remoteconf.h: free spspec
when deleting a rmconf struct. patch by Roman Hoog Antink
<rha@open.ch>
* src/racoon/: remoteconf.c, session.c: fixed some memory leaks in
remoteconf. patch by Roman Hoog Antink <rha@open.ch>
* src/racoon/: cfparse.y, prsa_par.y: fixed some memory leaks
during configuration parsing. patch by Roman Hoog Antink
<rha@open.ch>
2011-03-01 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/: isakmp.c, pfkey.c: plog text fixes, patch from M E
Andersson <debian@gisladisker.se>
* src/racoon/cfparse.y: reset yyerrorcount before doing parse
stuff. patch by Roman Hoog Antink <rha@open.ch>
2011-02-20 Timo Teras <timo.teras@iki.fi>
* src/racoon/oakley.c: From Roman Hoog Antink <rha@open.ch>: Fix
memory leak when using plain RSA key authentication.
2011-02-11 Timo Teras <timo.teras@iki.fi>
* src/racoon/plainrsa-gen.c: From Mats E Andersson
<debian@gisladisker.se>: Fix fprintf format specifier usage from
previous patch.
2011-02-10 Timo Teras <timo.teras@iki.fi>
* src/racoon/plainrsa-gen.c: From Mats Erik Andersson
<debian@gisladisker.se>: Implement importing of RSA keys from PEM
files.
* src/racoon/prsa_par.y: From M E Andersson
<debian@gisladisker.se>: Fix parsing of restricted RSA key
addresses.
2011-02-02 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/: cftoken.l, isakmp.c, remoteconf.h, sainfo.c,
sainfo.h: store ph1id in an u_int32_t instead of a (signed)int.
Patch from Christophe Carre
2011-01-28 Timo Teras <timo.teras@iki.fi>
* src/racoon/: sainfo.c, sainfo.h, session.c: From Roman Hoog
Antink <rha@open.ch>: Clean up sainfo reloading: rename the
functions, and remove unneeded global variable.
* src/racoon/: remoteconf.c, remoteconf.h, session.c: From Roman
Hoog Antink <rha@open.ch>: Clean up rmconf reloading: rename the
functions, and remove unneeded global variable.
* src/racoon/plog.c: From Roman Hoog Antink <rha@open.ch>: Log
remote IP address if available (slightly modified by tteras)
2011-01-22 Timo Teras <timo.teras@iki.fi>
* src/racoon/isakmp_inf.c: From Roman Hoog Antink <rha@open.ch>:
Fixes a null pointer dereference that might occur after removing
peers from the config and then reloading.
2011-01-20 Yvan Vanhullebus <vanhu@netasq.com>
* src/libipsec/pfkey.c: fixed a typo, it will now compile when
KMADDRESS is defined. reported by Roman Hoog Antink (rha (at)
open.ch)
2010-12-28 Timo Teras <timo.teras@iki.fi>
* src/racoon/handler.c: From Roman Hoog Antink <rha@open.ch>: Fix
config reload to not delete too many phase 2 handles, because wrong
chain field is used when enumerating the handles.
2010-12-16 gdt
* src/racoon/oakley.c: When encountering a certificate where "ID
mismatched with ASN1 SubjectName", and verify_identifier is off,
don't raise an error. This makes the behavior match the man page.
Patch sent for review long ago:
http://mail-index.netbsd.org/tech-security/2006/03/24/0000.html
with no negative feedback received to date.
2010-12-14 Timo Teras <timo.teras@iki.fi>
* src/racoon/ipsec_doi.c: From Roman Hoog Antink <rha@open.ch>: Fix
possible null derefence.
2010-12-08 Timo Teras <timo.teras@iki.fi>
* src/racoon/admin.c: Use separate SA addresses for phase2's
created by admin command. The phase2 startup overwrites src/dst with
ISAKMP ports if they are zero and we don't want that to happen for
the SA ports.
2010-12-08 joerg
* src/libipsec/pfkey.c: ANSIfy
2010-12-07 Timo Teras <timo.teras@iki.fi>
* src/racoon/isakmp_quick.c: Fix spacing and improve wording in
some log messages.
2010-12-03 Timo Teras <timo.teras@iki.fi>
* src/libipsec/ipsec_dump_policy.c: Recognize direction for Linux
per-socket policies.
* src/: libipsec/libpfkey.h, libipsec/pfkey_dump.c, setkey/parse.y,
setkey/setkey.8: Support GRE key as upper layer protocol
specifier (will be supported in Linux kernel 2.6.38).
* src/racoon/grabmyaddr.c: Netlink deletion notification does not
guarentee actual address deletion: it might still exist on some
other interface. Make sure we do not unbind unless the address is
really gone.
2010-11-17 Timo Teras <timo.teras@iki.fi>
* src/racoon/: handler.c, handler.h, isakmp.c, isakmp_inf.c: Fix my
previous patch to not call purge_remote() twice. Change the place
where purge_remote() is called. This fixes also a possible crash
from the same patch since ph1->remote can be NULL (when we are
responder and config is not yet selected).
2010-11-12 Timo Teras <timo.teras@iki.fi>
* src/racoon/: admin.c, isakmp.c, isakmp_var.h, pfkey.c:
isakmp_post_acquire is now called from admin commands too, add a
flag so admin commands can be used to establish even passive links
on demand.
* src/racoon/isakmp.c: Purge all IPsec-SA's if the last main
ISAKMP-SA for the node is deleted by remote request and the phase1
rekeying is enabled (this will also trigger the new phase1_dead
script hook).
* src/racoon/: handler.h, isakmp_inf.c: Improve DPD sequence checks
to allow any reply within valid sequence window to be proof of
livelyness. This can improves things if there's random packet
delays, or if racoon is not getting enough CPU time.
* src/racoon/: admin.c, admin.h, kmpstat.c, racoonctl.c: Extern
admin protocol to allow reply packets to exceed 64kb. E.g SA dumps
with many established SAs can be easily over the limit.
2010-10-22 Timo Teras <timo.teras@iki.fi>
* src/racoon/grabmyaddr.c: Change Linux Netlink address monitoring
to monitor local route changes. This works around a kernel bug, and
slightly improves behaviour on some special cases.
2010-10-21 Timo Teras <timo.teras@iki.fi>
* src/racoon/: admin.c, evt.c, grabmyaddr.c, isakmp.c, pfkey.c,
session.c, session.h: Introduce priorities for file descriptor
polling mechanism and give priority to admin port. If admin port is
used by ISAKMP-SA hook scripts they should be preferred, other wise
heavy traffic can delay admin port requests considerably. This in
turn may cause renegotiation loop for ISAKMP-SA. This is mostly
useful for OpenNHRP setup, but can benefit other setups too.
* src/racoon/: admin.c, handler.c, handler.h: Remove
initial-contact entry when all ISAKMP-SA are purged via adminport.
This will avoid stale security associations if some of the delete
notifications happens to get lost.
2010-10-20 Timo Teras <timo.teras@iki.fi>
* src/racoon/crypto_openssl.c: Use high-level openssl EVP and HMAC
functions when possible: this allows openssl to perform hardware
acceleration if available.
* src/racoon/: isakmp.c, isakmp_quick.c: Various improvements to
error log messages and a few additional error log messages to
improve diagnosing an error condition.
* src/racoon/grabmyaddr.c: Fix address comparison so we actually
close sockets which were bound to IP-address that got deconfigured.
2010-10-11 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/ipsec_doi.c: report a higher encryption key length in
approval for OBEY / CLAIM / STRICT modes
2010-09-27 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/isakmp_xauth.c: fixed some typos in logs (reported by
fazaeli (at) sepehrs.com)
2010-09-24 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/cftoken.l: fixed a fd leak, patch by getlaser (at)
gmail.com
2010-09-22 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/admin.c: get the correct length of username when
processing ADMIN_LOGOUT_USER, patch by rweikusat (at) mssgmbh.com
* src/racoon/nattraversal.h: fixed a typo in macros, reported by
marisp (at) mt.lv
2010-09-21 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/isakmp_cfg.c: moved from utmp.h to utmpx.h (patch
provided by marcin.cieslak (at) gmail.com)
2010-09-08 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/remoteconf.c: fixed remoteconf selection when no ID
specified in configuration, and added some debug to remoteconf
selection
2010-08-26 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/remoteconf.c: fix by Sergio.Gelato (at) astro.su.se:
duplicate some dynamic values in duprmconf()
2010-08-04 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/isakmp_cfg.c: fixed answer for IP4_SUBNET request
2010-07-30 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/doc/FAQ: updated link to NetBSD's documentation
2010-06-22 Thomas Klausner <wiz@netbsd.org>
* src/racoon/racoon.conf.5: Bump date for previous.
2010-06-22 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/: cfparse.y, cftoken.l, isakmp.c, isakmp_inf.c,
racoon.conf.5, remoteconf.c, remoteconf.h: added a specific
script hook when a dead peer is detected
2010-06-04 Thomas Klausner <wiz@netbsd.org>
* src/setkey/setkey.8: New sentence, new line. Bump date for
previous.
2010-06-04 Yvan Vanhullebus <vanhu@netasq.com>
* src/setkey/: parse.y, setkey.8, token.l: Added support for
spdupdate command in setkey
2010-04-07 Yvan Vanhullebus <vanhu@netasq.com>
* src/libipsec/ipsec_strerror.c: by Eric Preston: fixed a typo
2010-04-02 Christos Zoulas <christos@netbsd.org>
* src/: libipsec/pfkey_dump.c, racoon/backupsa.c: handle ctime
returning NULL.
2010-03-11 Christos Zoulas <christos@netbsd.org>
* src/racoon/handler.c: PR/42363: Yasuoka Masahiko: Second part of
the patch: iterate only on the phase2 handles that are bound by the
given phase1 handle.
2010-03-05 Timo Teras <timo.teras@iki.fi>
* src/: libipsec/ipsec_set_policy.3, racoon/privsep.c,
racoon/doc/FAQ, setkey/setkey.8: From Stefan Bauer: Fix multiple
typoes and manpage formatting errors.
2010-03-04 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/session.c: From Pierre POMES: fixed admin port
initialization
2010-02-28 snj
* src/racoon/: sockmisc.c, sockmisc.h: Fight the ever-increasing
size of src checkouts by spelling "useful" without an extra l.
2010-02-09 Thomas Klausner <wiz@netbsd.org>
* src/racoon/: pfkey.c, proposal.h: Fix typo in comment.
2010-01-17 Thomas Klausner <wiz@netbsd.org>
* src/racoon/sainfo.c: Free strdeupped string after using it. Found
by cppcheck.
* src/racoon/: eaytest.c, ipsec_doi.c: Close file handles after
using them. Found by cppcheck.
2010-01-15 joerg
* src/setkey/setkey.8: Use .%U instead of .%O for URLs.
2009-12-11 Timo Teras <timo.teras@iki.fi>
* src/racoon/Makefile.am: From Paul Wernau: vmbuf.h was defined
twice in the headers. Remove the redundant entry so new install tool
does not complain about overwriting just installed file.
2009-11-22 Christos Zoulas <christos@netbsd.org>
* src/racoon/handler.c: PR/42363: Yasuoka Masahiko:
racoon uses a wrong IPsec-SA handle that is for other peer in case
it receives a ISAKMP message for IPsec-SA that has the same
message-id as the message-id that is received before.
racoon uses message-id to find the handle of IPsec-SA. The
message-id is a unique number for each peer, but different peers may
use the same value.
Different Windows Vista or Windows 7 peers seem to use the same
message-id. racoon can handle the first Windows's Phase-2, but it
cannot handle the second Windows. Because racoon misunderstands the
message for the second Windows as the message for the first Windows.
>Category: bin >Synopsis: racoon uses a wrong IPsec-SA
that is for different peer >Confidential: no >Severity:
serious >Priority: medium >Responsible: bin-bug-people
>State: open >Class: sw-bug >Submitter-Id: net
>Arrival-Date: Sun Nov 22 18:25:00 +0000 2009 >Originator:
yasuoka@iij.ad.jp
2009-10-29 Christos Zoulas <christos@netbsd.org>
* src/setkey/token.l: use %option noinput nounput
2009-10-28 Christos Zoulas <christos@netbsd.org>
* src/setkey/token.l: no unput
2009-10-14 joerg
* src/libipsec/ipsec_set_policy.3: Do not use .Xo/.Xc to workaround
ancient groff limits.
* src/setkey/setkey.8: Do not use .Xo/.Xc to work around ancient
groff limits. Fix markup.
* src/racoon/racoon.conf.5: Don't use .Xo/.Xc to work around
ancient groff limits. Set only one list type.
2009-09-18 Timo Teras <timo.teras@iki.fi>
* src/racoon/: isakmp_agg.c, isakmp_ident.c: From Tomas Mraz: Fix
gssapi error checking.
2009-09-03 Timo Teras <timo.teras@iki.fi>
* src/racoon/: admin.c, handler.c, handler.h, isakmp.c,
isakmp_var.h, pfkey.c: When rekeying phase2 use phase1 used to
negotiate phase2 as a hint to select the phase1 for rekeying the new
phase2.
2009-09-01 Timo Teras <timo.teras@iki.fi>
* src/racoon/: nattraversal.c, racoon.conf.5, vendorid.c: Check
nat_traversal configuration from remote configuration candidates
when acting as responder. Enable NAT-T if any of the remote
candidates have NAT-T enabled.
* src/racoon/remoteconf.c: Change remote conf matching level to
matching score. This way one can override anonymous certificate
block config with more exact "inhereted" IP specific block.
* src/racoon/: isakmp.c, racoon.conf.5: From Maik Broemme: export
ISAKMP SA identity as REMOTE_ID for phase1 up script (trac #313).
2009-08-24 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/oakley.c: fixed typo: algoriym -> algorithm
2009-08-19 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/remoteconf.c: fixed address check in
rmconf_match_type(), just check address with wildcard port
2009-08-19 Timo Teras <timo.teras@iki.fi>
* src/racoon/remoteconf.c: Have an enum for rmconf_match_type()
return values to make the code a bit more readable.
2009-08-18 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/oakley.c: typo: algoritym -> algorithm
2009-08-17 Yvan Vanhullebus <vanhu@netasq.com>
* src/libipsec/libpfkey.h: do not use SADB_X_NAT_T_NEW_MAPPING to
check system support for NAT-T, as at least FreeBSD doesn't have
this define anymore
* src/racoon/schedule.h: include stddef.h so we have a chance to
get the system offsetof if present
* src/racoon/crypto_openssl.h: removed a self include
2009-08-13 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/oakley.c: fixed a potential DoS in
oakley_do_decrypt(), reported by Orange Labs
2009-08-10 Timo Teras <timo.teras@iki.fi>
* src/racoon/pfkey.c: Don't print EAGAIN error from
pfkey_handler(), it can occur normally under some code paths and is
not a hard error in any case.
2009-08-06 Timo Teras <timo.teras@iki.fi>
* src/setkey/setkey.c: From Paul Wenau: Check fgets return value in
setkey to make gcc happy.
2009-08-05 Timo Teras <timo.teras@iki.fi>
* src/racoon/pfkey.c: From Paul Wernau: Fix transport mode per-port
security associations that got broke during NAT-T fixes.
2009-07-07 Timo Teras <timo.teras@iki.fi>
* src/racoon/sockmisc.c: From Arnaud Ebalard: Fix possible usage of
uninitialized local variable (not sure if any code path triggers
this, but this makes compiler happy).
2009-07-03 Timo Teras <timo.teras@iki.fi>
* src/racoon/: admin.c, grabmyaddr.c, handler.c, handler.h,
isakmp.c, isakmp_cfg.c, isakmp_inf.c, isakmp_quick.c,
nattraversal.c, pfkey.c, policy.c, remoteconf.c, remoteconf.h,
sockmisc.c, sockmisc.h, throttle.c: Get rid of the evil CMPSADDR
macro. Trac #295.
* src/: libipsec/libpfkey.h, libipsec/pfkey.c, racoon/isakmp.c,
racoon/isakmp_inf.c, racoon/pfkey.c, racoon/pfkey.h: From Yvan
Vanhullebus: Use SADB_X_EXT_NAT_T_* consistently for passing the
NAT-T port information. This might break compatibility with some
kernels, but as discussed this is the proper way to pass NAT-T ports
and the broken kernels need to be fixed.
2009-06-24 Timo Teras <timo.teras@iki.fi>
* src/racoon/session.c: Fix a call to null pointer: in some cases,
the unmonitor_fd can be called from another fd's callback. That
could lead to still have callback pending after unmonitoring the fd
resulting in a call to null pointer. This is fixed by making
unmonitor_fd now clear the pending fd_set too. Bug was introduced
by my commit in 2008-12-23.
2009-05-20 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/isakmp.h: typo
2009-05-19 Timo Teras <timo.teras@iki.fi>
* src/racoon/: ipsec_doi.c, isakmp.c: From Jukka Salmi: Fix couple
of typos from previous commit.
2009-05-18 Timo Teras <timo.teras@iki.fi>
* src/racoon/: ipsec_doi.c, isakmp.c, sockmisc.c, sockmisc.h: From
Tomas Mraz: Introduce union sockaddr_any and use it to make code
more readable. Related to trac #293.
* src/racoon/isakmp_inf.c: From Tomas Mraz: Remove variable that is
not really used; only referenced while uninitialized causing
valgrind error.
* src/racoon/nattraversal.c: From Tomas Mraz: Fix natt_flags check.
2009-05-04 Thomas Klausner <wiz@netbsd.org>
* src/racoon/racoon.conf.5: Remove superfluous spaces around
parentheses.
2009-04-29 Timo Teras <timo.teras@iki.fi>
* src/racoon/crypto_openssl.c: From Ross Meng: Fix a memory leak in
X509 certificate validation.
2009-04-28 Timo Teras <timo.teras@iki.fi>
* src/racoon/handler.c: Reset nat_oa variables too when reusing
phase two handler. Otherwise phase2 rekeying might fail in some
scenarios.
2009-04-22 Timo Teras <timo.teras@iki.fi>
* src/racoon/isakmp_frag.c: From Neil Kettle: Fix a possible null
pointer dereference in fragmentation code.
2009-04-21 Timo Teras <timo.teras@iki.fi>
* src/racoon/: grabmyaddr.c, grabmyaddr.h, session.c: Fix
strict_address to work again. The lists needs to be initialized
before configuration is read, which happens before my_addr_init()
call.
2009-04-20 Timo Teras <timo.teras@iki.fi>
* src/racoon/: isakmp.c, isakmp.h, isakmp_var.h: Fix a memory leak
in certificate request generation.
* src/racoon/: isakmp_inf.c, isakmp_xauth.c, plog.c: Orignally from
Bin Li: Fix possible memory corruption in binsanitize().
* src/racoon/crypto_openssl.c: From Stephen Bevan: Fix a x509
signature verification memory leak.
* src/racoon/: admin.c, racoonctl.c: Originally from Bin Li: Fix a
crash with racoonctl logout user.
* src/racoon/nattraversal.c: Fix a memory leak in nat-t keepalive
code.
* src/racoon/handler.c: From Paul Moore: Phase2 message id's should
be unique wrt phase1, not globally.
2009-03-13 Timo Teras <timo.teras@iki.fi>
* src/racoon/: pfkey.c, remoteconf.h: From Arnaud Ebalard: Fix
couple of problems with previous commit.
2009-03-12 he
* src/racoon/: isakmp.c, remoteconf.c: When casting to/from a
pointer to an integral type (a bad practice, if you ask me), you
need to cast via intptr_t for portability.
2009-03-12 Thomas Klausner <wiz@netbsd.org>
* src/racoon/racoon.conf.5: New sentence, new line. Avoid marking
up punctuation.
* src/racoon/racoonctl.8: Bump date for previous. Sort options to
establish-sa. Stop using Xo/Xc.
2009-03-12 Timo Teras <timo.teras@iki.fi>
* src/racoon/: admin.c, cfparse.y, cftoken.l, crypto_openssl.c,
crypto_openssl.h, dnssec.c, dnssec.h, handler.c, handler.h,
ipsec_doi.c, ipsec_doi.h, isakmp.c, isakmp.h, isakmp_agg.c,
isakmp_base.c, isakmp_ident.c, isakmp_inf.c, isakmp_quick.c,
isakmp_var.h, nattraversal.c, oakley.c, oakley.h, racoon.conf.5,
racoonctl.8, racoonctl.c, remoteconf.c, remoteconf.h, sockmisc.c,
vendorid.c: Support multiple anonymous remotes and decide
remoteconf based on identity, received certificates and other
information. General code clean up.
2009-03-06 Timo Teras <timo.teras@iki.fi>
* src/setkey/: extern.h, parse.y, setkey.c: setkey: fix deleteall
in Linux
Linux requires SADB_DELETE message to have SPI. So send a
SADB_DELETE message for each matching SA. Trac #284.
From: Gabriel Somlo <somlo@cmu.edu>
2009-02-16 Timo Teras <timo.teras@iki.fi>
* src/libipsec/policy_parse.y: From Paul Moore: Fix a heap
corruption bug (yacc return non-null terminated buffer and sprintf
writes over bounds).
2009-02-11 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/: isakmp.c, sockmisc.c, sockmisc.h: trac#301: fixed
IPsec SAs flush in purge_remote() when NAT-T enabled but no NAT-T on
tunnel
2009-02-03 Timo Teras <timo.teras@iki.fi>
* src/racoon/isakmp.c: From: Phil Sutter. Fix script environment
variables with IPv6 addresses.
2009-01-26 Timo Teras <timo.teras@iki.fi>
* src/racoon/main.c: Argument parsing needs lcconf initialized.
2009-01-24 Thomas Klausner <wiz@netbsd.org>
* src/racoon/racoonctl.c: Sort options in usage.
* src/racoon/racoonctl.8: Sort options. New sentence, new line.
* src/racoon/racoon.8: Sort options.
2009-01-23 Timo Teras <timo.teras@iki.fi>
* src/racoon/: racoonctl.8, racoonctl.c: Update usage and manpage
for racoonctl.
* src/racoon/: main.c, racoon.8: Racoon -v to print version and
compilation information. Update usage message.
* NEWS: Update NEWS with major changes since 0.7 release.
* src/racoon/schedule.c: Fix monotonic scheduler change, to not
refresh 'now' before exit. Otherwise we can return negative timeout
after spending time handling other events.
* src/racoon/: handler.c, pfkey.c: From Arnaud Ebalard: Handle
reception of MIGRATE message during Phase 1 and Phase 2 negotiation.
Also corrects some debugging statements.
* src/racoon/pfkey.c: From Arnaud Ebalard: On the responder (for
instance), there is a need to not only migrate local and remote
addresses of Phase 1 that match previous addresses but also the
local and remote addresses of a Phase 1 *associated* with a migrated
Phase 2. For instance, we have that need when receiving the first
MIGRATE/KMADDRESS message because the old addresses are still the
HoA and the address of the HA (while the peer has contacted us using
the CoA and we have negotiated this address as src attribute in
Phase 2). The patch fixes that by having migrate_ph1_ike_addresses()
called from migrate_ph2_ike_addresses() callback.
* src/racoon/isakmp_quick.c: From Arnaud Ebalard: Set phase2 spid
when acting as responder.
* configure.ac, src/racoon/handler.c, src/racoon/handler.h,
src/racoon/isakmp_inf.c, src/racoon/isakmp_xauth.c,
src/racoon/schedule.c, src/racoon/schedule.h,
src/racoon/throttle.c, src/racoon/throttle.h: Detect if monotonic
system clock is available, and use it for relative time measurements
to avoid complite hang if time jumps backwards.
* src/racoon/: cfparse.y, ipsec_doi.c, isakmp.c, isakmp_agg.c,
isakmp_base.c, isakmp_cfg.c, isakmp_ident.c, isakmp_xauth.c,
oakley.c, oakley.h: Fix authentication method ambiguity by
internally using unique ID and setting/interpreting the wire format
based on received vendor ID:s. Fixes trac #280.
* src/racoon/: handler.h, isakmp_agg.c, isakmp_base.c,
isakmp_ident.c, vendorid.c, vendorid.h: Introduce vendorid
bitmask that can be used otherwhere to detect peer capabilities.
* configure.ac, src/racoon/admin.c, src/racoon/evt.c,
src/racoon/grabmyaddr.c, src/racoon/isakmp.c, src/racoon/pfkey.c,
src/racoon/session.c, src/racoon/session.h: Remove "fastquit"
configure option and make it the default behaviour. The previous
normal behaviour is buggy, as after flush kernel can immediately
create larval SA:s which would prevent exit.
2009-01-20 Timo Teras <timo.teras@iki.fi>
* Makefile.am, misc/cvs2cl.pl, misc/cvsusermap: Autogenerate
ChangeLog from NetBSD CVS. Put sourceforge.net changes to
ChangeLog.old.
2009-01-10 Thomas Klausner <wiz@netbsd.org>
* src/racoon/racoon.conf.5: Make ready for HTML output. Use proper
escape for backslash ('\e').
2009-01-10 Timo Teras <timo.teras@iki.fi>
* src/racoon/: crypto_openssl.c, racoon.conf.5: From Cyrus Rahman:
Accept RFC2253 compliant escaped special characters for asn1dn
identifier.
2009-01-09 Timo Teras <timo.teras@iki.fi>
* configure.ac: Fix a CPPLAGS typo to CPPFLAGS which was intended
2009-01-05 Timo Teras <timo.teras@iki.fi>
* src/racoon/: cfparse.y, cftoken.l, racoon.conf.5: Remove obsolete
configuration options, fix radius configuration block and add GRE as
recognized protocol.
* src/racoon/session.c: Do not use counting in signal handling as
it was unsafe by not using atomic functions (post increment is not
necessarily atomic). Instead reap all children on SIGCHLD as that
was the only signal needing signal counting.
2008-12-30 Timo Teras <timo.teras@iki.fi>
* src/racoon/session.c: schedular() call can now modify fd mask so
make the working copy just before calling select(); otherwise it can
contain bad file descriptors
2008-12-29 Michael van Elst <mlelstv@netbsd.org>
* src/setkey/parse.y: support icmp codes. Fixes PR 39056.
2008-12-24 Christos Zoulas <christos@netbsd.org>
* src/racoon/grabmyaddr.c: remove sin{6,}_len linux does not have
it. From Timo Teras.
* src/racoon/grabmyaddr.c: I was wrong. addr is actually set.
* src/racoon/grabmyaddr.c:
- make this compile by zeroing out the whole structure not just
bogus fields.
- set length field of sockets appropriately.
- mark bogus no-op code (I don't understand what the author intended
here).
2008-12-23 Thomas Klausner <wiz@netbsd.org>
* src/racoon/racoon.conf.5: Bump date for identity configuration
option removal.
2008-12-23 Timo Teras <timo.teras@iki.fi>
* src/racoon/: cfparse.y, cftoken.l, ipsec_doi.c, localconf.c,
localconf.h, racoon.conf.5: Remove the obsoleted global identity
configuration option.
* src/racoon/: admin.c, admin_var.h, cfparse.y, debug.h, evt.c,
evt.h, grabmyaddr.c, grabmyaddr.h, handler.c, isakmp.c,
isakmp_inf.c, isakmp_var.h, localconf.c, localconf.h, main.c,
nattraversal.c, pfkey.c, pfkey.h, privsep.c, session.c,
session.h: rewrite local address detection make some functions
static that arr not needed globally rework how fd_set is
construction for the main loop select()
2008-12-18 Timo Teras <timo.teras@iki.fi>
* src/racoon/pfkey.c: From Arnaud Ebalard: Delete larval ph2handles
when expire with hard lifetime received
2008-12-16 Timo Teras <timo.teras@iki.fi>
* README: Update README
* src/racoon/pfkey.c: Fix transport mode address selection in
acquire handling. Some earlier fixes got lost on 2008-12-05 commit.
2008-12-11 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/grabmyaddr.c: Fixed compilation on FreeBSD (RTM_IFINFO
and RTM_OIFINFO stuff)
* src/racoon/isakmp.c: Fixed compilation when DPD support is
disabled
2008-12-08 Timo Teras <timo.teras@iki.fi>
* src/racoon/: pfkey.c, privsep.c, privsep.h: Do not cache pfkey
sockets: it might cause to not handle some pfkey events when
select() has marked pfkey socket readable, but a timer callback
first calls pfkey_dump_sadb().
2008-12-05 Timo Teras <timo.teras@iki.fi>
* src/: libipsec/key_debug.c, libipsec/libpfkey.h,
libipsec/pfkey.c, racoon/handler.c, racoon/handler.h,
racoon/ipsec_doi.c, racoon/isakmp.c, racoon/isakmp_quick.c,
racoon/pfkey.c, racoon/policy.c, racoon/policy.h: From Arnaud
Ebalard: Improved Mobile IPv6 support per
draft-ebalard-mext-pfkey-enhanced-migrate.
2008-12-04 Christoph Badura <bad@netbsd.org>
* src/racoon/privsep.c: Fix typo in previous and use SIG_IGN as I
intended.
2008-12-02 Timo Teras <timo.teras@iki.fi>
* src/racoon/session.c: Explicitly ignore SIGPIPE. Default action
on Linux is terminate.
2008-11-28 Thomas Klausner <wiz@netbsd.org>
* src/racoon/racoon.conf.5: Remove empty line. Fix typo. New
sentence, new line.
2008-11-27 Yvan Vanhullebus <vanhu@netasq.com>
* src/racoon/main.c: Set up a default value for Mode Config Pool
size if pool address specified but pool size not specified
* src/racoon/isakmp_cfg.c: Fixed pool resizing
2008-11-27 Timo Teras <timo.teras@iki.fi>
* src/racoon/pfkey.c: From Arnaud Ebalard: Remove MAXNESTEDSA
weirdness. It's probably meant for bundle support which is not done.
When someone actually writes bundle support, the nested SA stuff
would probably be reworked too anyway.
* src/: libipsec/libpfkey.h, libipsec/pfkey.c, racoon/cfparse.y,
racoon/cftoken.l, racoon/localconf.c, racoon/localconf.h,
racoon/pfkey.c, racoon/racoon.conf.5: From: Matthew Krenzer
Ability to set pfkey socket buffer size via configuration file
directive. (Indentation and minor fixes by me.)
2008-11-25 Christoph Badura <bad@netbsd.org>
* src/racoon/: evt.c, privsep.c, session.c: Avoid using
MSG_NOSIGNAL as it is not available everywhere. Ignore SIGPIPE
instead.
* src/racoon/grabmyaddr.c: Ignore unspecified and looback
addresses. Ignoring unspecified addresses prevents racoon from
trying to bind to the wildcard address and specific addresses
simultaneously after e.g. dhclient has changed an interface's
address to 0.0.0.0.
* src/racoon/grabmyaddr.c: RTM_DELETE and RTM_IFINFO don't carry
info for added or deleted addresses. Ignore them silently.