summaryrefslogtreecommitdiff
path: root/usr.bin/sendbug
AgeCommit message (Collapse)Author
2010-04-20Fix some pclose() calls, spotted by Peter Philipp. ok deraadt@Stuart Henderson
2010-04-20Include usbdevs -v output in the PR template, showing us device IDs whichStuart Henderson
cannot be deduced from the dmesg output. Suggested by miod@, ok deraadt@
2010-03-23fixes memory and FILE handle leaks. `acpidir' is allocated viaTheo de Raadt
asprintf(3) and `ifp' is opened via popen(3), but not closed. from zinovik
2009-08-26initialize tmpdir before use; PR 6214Theo de Raadt
2009-06-10no real need for these in SEE ALSO; ok rayJason McIntyre
while here, tweak -D
2009-06-07Remove gecos expanding code, since the expanded string is no longerRay Lai
used.
2009-06-06Include pcidump and acpidump output when sendbug is run as root,Ray Lai
to aid ACPI development. Output files are base64 encoded in the report. Add a -E flag to decode files from a report to the current directory. Requested by deraadt, "Sweet!" marco, "Looks good to me." pirofti
2009-01-28Re-add "Category" category to sendbug template. Without it, reportsRay Lai
were categorized as "pending" and the report was never acked. Reported by RD Thrush and bernd@, OK miod
2008-12-14Remove useless fields from template, they just serve to confuseRay Lai
users. Idea okayed by art and deraadt, feedback from jmc.
2008-10-06parse all arguments before going and doing the job requested by -PTheo de Raadt
(slight change made to check argc == 0) from Daniel Dickman
2008-06-14- remove ports category after discussion w/ manyPeter Valchev
- sync architectures list with reality and fix a few names ok theo
2008-05-08replace "/usr/sbin/sendmail" with _PATH_SENDMAILCharles Longeau
no binary change found by Daniel Dickman <didickman at gmail.com> ok ray@
2008-04-19Decrement len variable after removing newline, prevents copying theRay 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-04Eliminate false positives from <> comments, avoiding manglingRay 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-03In 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) byTheo 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-18Don't treat lines in angle brackets <...> as comments in dmesg.Ray Lai
Discovered and tested by deanna.
2007-09-07Synced atomicio implementation in nc and sendbug with ssh.Tobias Stoeckmann
OK djm@, joris@, ray@
2007-07-31Remove some unused headers, from Igor Zinovik.Ray Lai
2007-05-31convert to new .Dd format;Jason McIntyre
2007-05-11If fork fails with EAGAIN, ignore errno and fail. Looping justRay Lai
worsens the problem. cloder@ and deraadt@ agree.
2007-05-11Instead of returning -1 and setting errno to ECHILD if the editorRay 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-09Be more consistent about when dmesg is attached to template.Ray Lai
Specifically, -V.
2007-05-07Simplify some loops.Ray Lai
OK jaredy@ and moritz@.
2007-05-06Restore signals before returning.Ray Lai
2007-05-06Save and restore signals once instead of each time fork fails withRay Lai
EAGAIN.
2007-04-25Remove 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-13document dmesg(8) attachment a little more fully; ok rayJason McIntyre
2007-04-13+.Xr dmesg 8Jason McIntyre
2007-04-07Sprinkle const poison, remove unneeded variable.Ray Lai
2007-04-07Use __progname and warn instead of fprintf where appropriate.Ray Lai
2007-04-07De-lint.Ray Lai
2007-04-06Warn if one of the required fields is not filled out. RequestedRay Lai
by millert@.
2007-04-06Don't shadow variables. Enable -Wshadow.Ray Lai
2007-04-06argc does not include __progname after subtracting optind.Ray Lai
2007-04-06Only include newest dmesg.Ray Lai
``Come on, take some risks'' deraadt@.
2007-04-06Attach dmesg to sendbug reports by default. Use -D flag to disable.Ray Lai
OK deraadt@ and millert@.
2007-03-28flesh the text out; ok rayJason McIntyre
2007-03-28+.Xr crash 8Jason McIntyre
2007-03-28CC: a copy of the bug report to sender. Requested by Travers Buda.Ray Lai
OK millert@.
2007-03-27Replace some scary pointer code. The new code is O(n^2) for stringsRay 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-27Remove variable name from prototype.Ray Lai
2007-03-26- tweak the opening sentenceJason 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-26Simplify gecos parsing code and prevent buffer overflowMoritz Jodeit
when gecos field contains multiple '&' characters with other characters inbetween. With and OK ray@
2007-03-26Remove pointless cast. No binary change.Ray Lai
2007-03-26Break on waitpid success; we shouldn't continue on WIFSIGNALED, andRay Lai
WIFCONTINUED can't happen in this case. OK deraadt@.
2007-03-26Certain errors in editit() should be fatal.Ray Lai
2007-03-26Don't print warnings in editit(), instead make errno reliable onRay Lai
failure. If the editor exits non-zero, errno is set to ECHILD. OK deraadt@.
2007-03-26Accept $VISUAL in addition to $EDITOR.Ray Lai
2007-03-26Save and restore signal handlers. Although we do not install ourRay Lai
own signal handlers, this allows this code to be more easily reused in other programs. OK deraadt@