| Age | Commit message (Collapse) | Author | |
|---|---|---|---|
| 2008-10-06 | parse all arguments before going and doing the job requested by -P | Theo de Raadt | |
| (slight change made to check argc == 0) from Daniel Dickman | |||
| 2008-06-14 | - remove ports category after discussion w/ many | Peter Valchev | |
| - sync architectures list with reality and fix a few names ok theo | |||
| 2008-05-08 | replace "/usr/sbin/sendmail" with _PATH_SENDMAIL | Charles Longeau | |
| no binary change found by Daniel Dickman <didickman at gmail.com> ok ray@ | |||
| 2008-04-19 | Decrement len variable after removing newline, prevents copying the | Ray Lai | |
| NUL at the end of a string into the mail. Discovered by dasn. Move newline printing into outer loop, prevents multiple newlines from appearing if there are multiple comments in a line. Discovered by okan. OK okan. | |||
| 2008-01-04 | Eliminate false positives from <> comments, avoiding mangling | Ray Lai | |
| #include <bleh.h> in diffs: - Only match <> comments from the template. No other text surrounded by angle brackets is deleted. - Simplify code by using C strings. We no longer handle binary data, but I don't think binary data belongs in a bug report anyway. - Remove dmesg and header comment checks. "That's a nice approach." deraadt, "seems ok for me" chl. | |||
| 2008-01-03 | In preparation of eliminating <> comment false positives: | Ray Lai | |
| - Move <> comments in template to an array, for easier reference later. - Rename "comment" variable to avoid shadowing. - Change some <> comments to SENDBUG comments. OK chl@ | |||
| 2007-10-17 | - Be more careful about pre-existing SIGCHLD handlers (or SIG_IGN) by | Theo de Raadt | |
| temporarily restoring default behaviour. This is not 100% ideal. But this fixes editor handling in mail... bah, it is really unfortunate that got broken - refactor the restoration code as well, to make it simpler ok ray | |||
| 2007-09-18 | Don't treat lines in angle brackets <...> as comments in dmesg. | Ray Lai | |
| Discovered and tested by deanna. | |||
| 2007-09-07 | Synced atomicio implementation in nc and sendbug with ssh. | Tobias Stoeckmann | |
| OK djm@, joris@, ray@ | |||
| 2007-07-31 | Remove some unused headers, from Igor Zinovik. | Ray Lai | |
| 2007-05-31 | convert to new .Dd format; | Jason McIntyre | |
| 2007-05-11 | If fork fails with EAGAIN, ignore errno and fail. Looping just | Ray Lai | |
| worsens the problem. cloder@ and deraadt@ agree. | |||
| 2007-05-11 | Instead of returning -1 and setting errno to ECHILD if the editor | Ray Lai | |
| has a non-zero exit status, just return the exit status on success. Hard errors still return -1. If the editor is killed, return -1 and set EINTR. | |||
| 2007-05-09 | Be more consistent about when dmesg is attached to template. | Ray Lai | |
| Specifically, -V. | |||
| 2007-05-07 | Simplify some loops. | Ray Lai | |
| OK jaredy@ and moritz@. | |||
| 2007-05-06 | Restore signals before returning. | Ray Lai | |
| 2007-05-06 | Save and restore signals once instead of each time fork fails with | Ray Lai | |
| EAGAIN. | |||
| 2007-04-25 | Remove extra code that only applies to setuid/setgid programs. | Ray Lai | |
| ``Slacker! We'll burn your hands if you don't get going!'' deraadt@ | |||
| 2007-04-13 | document dmesg(8) attachment a little more fully; ok ray | Jason McIntyre | |
| 2007-04-13 | +.Xr dmesg 8 | Jason McIntyre | |
| 2007-04-07 | Sprinkle const poison, remove unneeded variable. | Ray Lai | |
| 2007-04-07 | Use __progname and warn instead of fprintf where appropriate. | Ray Lai | |
| 2007-04-07 | De-lint. | Ray Lai | |
| 2007-04-06 | Warn if one of the required fields is not filled out. Requested | Ray Lai | |
| by millert@. | |||
| 2007-04-06 | Don't shadow variables. Enable -Wshadow. | Ray Lai | |
| 2007-04-06 | argc does not include __progname after subtracting optind. | Ray Lai | |
| 2007-04-06 | Only include newest dmesg. | Ray Lai | |
| ``Come on, take some risks'' deraadt@. | |||
| 2007-04-06 | Attach dmesg to sendbug reports by default. Use -D flag to disable. | Ray Lai | |
| OK deraadt@ and millert@. | |||
| 2007-03-28 | flesh the text out; ok ray | Jason McIntyre | |
| 2007-03-28 | +.Xr crash 8 | Jason McIntyre | |
| 2007-03-28 | CC: a copy of the bug report to sender. Requested by Travers Buda. | Ray Lai | |
| OK millert@. | |||
| 2007-03-27 | Replace some scary pointer code. The new code is O(n^2) for strings | Ray Lai | |
| of '&', but gecos fields are usually short and most do not have '&' so the impact should be minimal. In return we get clearer and safer code. OK moritz@. | |||
| 2007-03-27 | Remove variable name from prototype. | Ray Lai | |
| 2007-03-26 | - tweak the opening sentence | Jason McIntyre | |
| - be less precise about editors in the opening blurb - note that prs are not confidential, and provide an alternative ok ray deraadt | |||
| 2007-03-26 | Simplify gecos parsing code and prevent buffer overflow | Moritz Jodeit | |
| when gecos field contains multiple '&' characters with other characters inbetween. With and OK ray@ | |||
| 2007-03-26 | Remove pointless cast. No binary change. | Ray Lai | |
| 2007-03-26 | Break on waitpid success; we shouldn't continue on WIFSIGNALED, and | Ray Lai | |
| WIFCONTINUED can't happen in this case. OK deraadt@. | |||
| 2007-03-26 | Certain errors in editit() should be fatal. | Ray Lai | |
| 2007-03-26 | Don't print warnings in editit(), instead make errno reliable on | Ray Lai | |
| failure. If the editor exits non-zero, errno is set to ECHILD. OK deraadt@. | |||
| 2007-03-26 | Accept $VISUAL in addition to $EDITOR. | Ray Lai | |
| 2007-03-26 | Save and restore signal handlers. Although we do not install our | Ray Lai | |
| own signal handlers, this allows this code to be more easily reused in other programs. OK deraadt@ | |||
| 2007-03-26 | Restore errno before calling perror(). | Ray Lai | |
| 2007-03-26 | Check fork() == -1 instead of fork() < 0. | Ray Lai | |
| 2007-03-26 | When someone hits ^C in an editor, the editor does exit(130), I | Ray Lai | |
| don't know why. Something to do with the shell, says deraadt@. Continuing to ignore these signals in the child fixes this. Now editit()'s return value can be reliably checked. OK deraadt@. | |||
| 2007-03-26 | more careful waitpid() discussed with ray (plus some knf) | Theo de Raadt | |
| 2007-03-25 | fork() does not return EPROCLIM, so don't test for it. | Ray Lai | |
| 2007-03-25 | more cleaning :) | Theo de Raadt | |
| 2007-03-25 | Change editit() to return -1 on error and 0 on success, like many | Ray Lai | |
| other C functions. (Currently this return value is unchecked.) | |||
| 2007-03-25 | KNF. | Ray Lai | |
| 2007-03-25 | Save errno before calling signal(). | Ray Lai | |
 |
index : src | |
| OpenBSD base system |
| summaryrefslogtreecommitdiff |