Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Restore beck's rev 1.3: snprintf() was reviewed | guenther | 2014-04-20 | 2 | -2/+2 |
| | |||||
* | More KNF. | jsing | 2014-04-20 | 4 | -6/+6 |
| | |||||
* | KNF. | jsing | 2014-04-20 | 12 | -1428/+1902 |
| | |||||
* | KNF. | jsing | 2014-04-20 | 12 | -940/+1184 |
| | |||||
* | KNF. | jsing | 2014-04-20 | 8 | -528/+718 |
| | |||||
* | More KNF. | jsing | 2014-04-20 | 26 | -160/+176 |
| | |||||
* | More KNF. | jsing | 2014-04-20 | 2 | -100/+124 |
| | |||||
* | gettimeofday() is portable enough and does not need a wrapper | deraadt | 2014-04-20 | 2 | -18/+4 |
| | |||||
* | calloc() rather than malloc+memset | deraadt | 2014-04-20 | 2 | -4/+4 |
| | |||||
* | Fix indentation, adding braces and combining a nested if to reduce depth | guenther | 2014-04-20 | 2 | -24/+28 |
| | |||||
* | KNF. | jsing | 2014-04-20 | 4 | -362/+438 |
| | |||||
* | KNF. | jsing | 2014-04-20 | 8 | -796/+786 |
| | |||||
* | Chop off more SSLv2 tentacles and start fixing and noting y2038 issues. | guenther | 2014-04-20 | 17 | -118/+44 |
| | | | | | | | APIs that pass times as longs will have to change at some point... Bump major on both libcrypto and libssl. ok tedu@ | ||||
* | return after error instead of plowing ahead. noticed by mancha1 at zoho | tedu | 2014-04-20 | 2 | -0/+2 |
| | |||||
* | Use calloc(a,b) instead of malloc(a*b) + memset(a*b). I don't know if | deraadt | 2014-04-20 | 2 | -4/+2 |
| | | | | | | | | | | | | | | | | | | | | | | this instance is integer-overflowable, but we cannot keep hand-auditing every instance (or apathetically ignoring these issues) when the simple calloc idiom is better in the presence of a good calloc(). It is simply unfeasible to always enter correct range checks before the aggregate size calculation, just go find some 4000 lines of code, REPAIR THEM ALL, then come back and tell me I am wrong. This only works on systems where calloc() does the integer overflow check, but if your system doesn't do this, you need to ask your vendor WHY THEY ARE 10 YEARS BEHIND IN BEST PRACTICE? This is the kind of problem that needs to be solved at the right layer. malloc integer-overflow was implicated in the 2002 OpenSSH hole. OpenSSH and much other code is now written to use calloc(), for instance OpenSSH has 103 calls to it. We feel safer with our use of calloc(). It is a natural approach for us to use calloc(). How safe do you feel on systems which lack that range check in their calloc()? Good writeup from 2006: http://undeadly.org/cgi?action=article&sid=20060330071917 | ||||
* | Undo a calloc() replacement. It lacks the integer overflow check that | deraadt | 2014-04-20 | 1 | -14/+1 |
| | | | | the system one has. | ||||
* | KNF. | jsing | 2014-04-20 | 24 | -2308/+2268 |
| | |||||
* | reset imprint to NULL to avoid double free. from mancha1 at zoho | tedu | 2014-04-20 | 2 | -0/+2 |
| | |||||
* | release buffers fix was lost in merge. put it back. | tedu | 2014-04-19 | 2 | -2/+4 |
| | |||||
* | change some rsh references to ssh. poke by jmc | tedu | 2014-04-19 | 2 | -9/+9 |
| | |||||
* | More KNF. | jsing | 2014-04-19 | 4 | -74/+84 |
| | |||||
* | More KNF. | jsing | 2014-04-19 | 6 | -90/+122 |
| | |||||
* | another attempt at fixing stale x509 data. since we don't know where the | tedu | 2014-04-19 | 2 | -20/+16 |
| | | | | | initial storage came from, we can't free it. just memset in the sequence case. probably ok beck | ||||
* | More KNF. | jsing | 2014-04-19 | 14 | -157/+163 |
| | |||||
* | More KNF. | jsing | 2014-04-19 | 2 | -40/+40 |
| | |||||
* | tweak previous; | jmc | 2014-04-19 | 1 | -3/+2 |
| | |||||
* | zap trailing whitespace; | jmc | 2014-04-19 | 1 | -2/+2 |
| | |||||
* | remove the openssl_fdset wrapper, and a variety of VMS'ism's like | deraadt | 2014-04-19 | 6 | -48/+19 |
| | | | | | crazy (void *) casts all over the place ok beck jsing | ||||
* | KNF. | jsing | 2014-04-19 | 12 | -1626/+1628 |
| | |||||
* | Remove a gethostbyname() cache layer. The internet works better these | deraadt | 2014-04-19 | 1 | -50/+1 |
| | | | | days. Initially fixed this, but Ted asked for it to die. | ||||
* | ReadFile() and GetStdHandle() are not very POSIX. | deraadt | 2014-04-19 | 4 | -45/+6 |
| | | | | ok beck jsing | ||||
* | truncation check some snprintf calls (over-cautiously in case) | deraadt | 2014-04-19 | 4 | -19/+78 |
| | | | | ok jsing beck | ||||
* | can't actually do this. cause unknown. | tedu | 2014-04-19 | 2 | -2/+2 |
| | |||||
* | Fix some serious pointer-arithmatic-magic-number-unchecked-return eyebleed | beck | 2014-04-19 | 1 | -53/+44 |
| | | | | | | | | | | that I stumbled into here and got stuck with. If modern society can get past selling daughters for cows, surely we can decide to write modern C code in an "application" that is probably 3 lines of shell/python/cgi away from talking to the internet in a lot of places.. (This file still needs a lot more love though) "oh god yuck" deraadt@ ok tedu@ | ||||
* | KNF. | jsing | 2014-04-19 | 12 | -1128/+1060 |
| | |||||
* | Repair some indent to be more KNF, and break instead of fallthrough to | deraadt | 2014-04-19 | 1 | -12/+14 |
| | | | | default which does break | ||||
* | one small tweak to avoid ever going off the end of a string. | tedu | 2014-04-19 | 1 | -4/+6 |
| | |||||
* | improved checking for invalid hashes. from solar designer | tedu | 2014-04-19 | 1 | -6/+10 |
| | |||||
* | More KNF. | jsing | 2014-04-19 | 6 | -66/+68 |
| | |||||
* | kill a stray \ | deraadt | 2014-04-19 | 1 | -2/+2 |
| | |||||
* | Lacking a proof that--for this implementation--exposure of Montgomery | guenther | 2014-04-19 | 4 | -14/+14 |
| | | | | | | | | multiplication or RSA blinding parameters doesn't permit retroactive timing analysis of the secrets, we'll do the stupidly cheap thing and cleanse them before freeing them. ok deraadt@ | ||||
* | More KNF. | jsing | 2014-04-19 | 10 | -476/+592 |
| | |||||
* | Remove hacky workaround for Cray T3E. | deraadt | 2014-04-19 | 1 | -11/+0 |
| | | | | ok guenther | ||||
* | More KNF. | jsing | 2014-04-19 | 8 | -522/+758 |
| | |||||
* | oops, typo got into change | deraadt | 2014-04-19 | 1 | -1/+1 |
| | |||||
* | More KNF. | jsing | 2014-04-19 | 6 | -20/+28 |
| | |||||
* | malloc + strlcpy -> strdup | deraadt | 2014-04-19 | 1 | -3/+2 |
| | |||||
* | unifdef ENOTDIR, everyone has it | deraadt | 2014-04-19 | 1 | -15/+3 |
| | |||||
* | We'll interpret a (void) cast on snprintf() to mean it's been verified that | guenther | 2014-04-19 | 55 | -147/+147 |
| | | | | | | truncation is either desirable, not an issue, or is detected and handled later ok deraadt@ | ||||
* | Use somewhat harsher language and better examples; demonstrate that | deraadt | 2014-04-19 | 4 | -109/+88 |
| | | | | | non-dangerous use functions is difficult. ok guenther |