git: ff2b1ffbb045 - main - bintrans: adjust mmencode.c to style(9)
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sun, 24 Apr 2022 06:24:23 UTC
The branch main has been updated by pstef:
URL: https://cgit.FreeBSD.org/src/commit/?id=ff2b1ffbb0454e23de374b0054b4bc06c81336a6
commit ff2b1ffbb0454e23de374b0054b4bc06c81336a6
Author: Piotr Pawel Stefaniak <pstef@FreeBSD.org>
AuthorDate: 2022-04-15 13:00:36 +0000
Commit: Piotr Pawel Stefaniak <pstef@FreeBSD.org>
CommitDate: 2022-04-24 06:19:49 +0000
bintrans: adjust mmencode.c to style(9)
---
usr.bin/bintrans/mmencode.c | 445 ++++++++++++++++++++++----------------------
1 file changed, 224 insertions(+), 221 deletions(-)
diff --git a/usr.bin/bintrans/mmencode.c b/usr.bin/bintrans/mmencode.c
index 511e470150bb..a391f4d7e1d4 100644
--- a/usr.bin/bintrans/mmencode.c
+++ b/usr.bin/bintrans/mmencode.c
@@ -1,52 +1,53 @@
/*
Copyright (c) 1991 Bell Communications Research, Inc. (Bellcore)
-Permission to use, copy, modify, and distribute this material
-for any purpose and without fee is hereby granted, provided
-that the above copyright notice and this permission notice
-appear in all copies, and that the name of Bellcore not be
-used in advertising or publicity pertaining to this
-material without the specific, prior written permission
-of an authorized representative of Bellcore. BELLCORE
-MAKES NO REPRESENTATIONS ABOUT THE ACCURACY OR SUITABILITY
-OF THIS MATERIAL FOR ANY PURPOSE. IT IS PROVIDED "AS IS",
+Permission to use, copy, modify, and distribute this material
+for any purpose and without fee is hereby granted, provided
+that the above copyright notice and this permission notice
+appear in all copies, and that the name of Bellcore not be
+used in advertising or publicity pertaining to this
+material without the specific, prior written permission
+of an authorized representative of Bellcore. BELLCORE
+MAKES NO REPRESENTATIONS ABOUT THE ACCURACY OR SUITABILITY
+OF THIS MATERIAL FOR ANY PURPOSE. IT IS PROVIDED "AS IS",
WITHOUT ANY EXPRESS OR IMPLIED WARRANTIES.
*/
+#include <stdbool.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>
-int PendingBoundary(s, Boundaries, BoundaryCt)
-char *s;
-char **Boundaries;
-int *BoundaryCt;
+static int
+PendingBoundary(char *s, char **Boundaries, int *BoundaryCt)
{
- int i, len;
+ int i;
+ size_t len;
- if (s[0] != '-' || s[1] != '-') return(0);
+ if (s[0] != '-' || s[1] != '-')
+ return (0);
-
- for (i=0; i < *BoundaryCt; ++i) {
- len = strlen(Boundaries[i]);
- if (!strncmp(s, Boundaries[i], len)) {
- if (s[len] == '-' && s[len+1] == '-') *BoundaryCt = i;
- return(1);
- }
- }
- return(0);
+ for (i = 0; i < *BoundaryCt; ++i) {
+ len = strlen(Boundaries[i]);
+ if (strncmp(s, Boundaries[i], len) == 0) {
+ if (s[len] == '-' && s[len + 1] == '-')
+ *BoundaryCt = i;
+ return (1);
+ }
+ }
+ return (0);
}
-static char basis_hex[] = "0123456789ABCDEF";
-static char index_hex[128] = {
- -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1,
- -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1,
- -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1,
- 0, 1, 2, 3, 4, 5, 6, 7, 8, 9,-1,-1, -1,-1,-1,-1,
- -1,10,11,12, 13,14,15,-1, -1,-1,-1,-1, -1,-1,-1,-1,
- -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1,
- -1,10,11,12, 13,14,15,-1, -1,-1,-1,-1, -1,-1,-1,-1,
- -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1
+#define basis_hex "0123456789ABCDEF"
+static const char index_hex[128] = {
+ -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1,
+ -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1,
+ -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1,
+ 0, 1, 2, 3, 4, 5, 6, 7, 8, 9,-1,-1, -1,-1,-1,-1,
+ -1,10,11,12, 13,14,15,-1, -1,-1,-1,-1, -1,-1,-1,-1,
+ -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1,
+ -1,10,11,12, 13,14,15,-1, -1,-1,-1,-1, -1,-1,-1,-1,
+ -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1
};
/* The following version generated complaints on Solaris. */
@@ -54,201 +55,203 @@ static char index_hex[128] = {
/* Since we're no longer ever calling it with anything signed, this should work: */
#define hexchar(c) (((c) > 127) ? -1 : index_hex[(c)])
-void toqp(infile, outfile)
-FILE *infile, *outfile;
+static void
+toqp(FILE *infile, FILE *outfile)
{
- int c, ct=0, prevc=255;
- while ((c = getc(infile)) != EOF) {
- if ((c < 32 && (c != '\n' && c != '\t'))
- || (c == '=')
- || (c >= 127)
- /* Following line is to avoid single periods alone on lines,
- which messes up some dumb smtp implementations, sigh... */
- || (ct == 0 && c == '.')) {
- putc('=', outfile);
- putc(basis_hex[c>>4], outfile);
- putc(basis_hex[c&0xF], outfile);
- ct += 3;
- prevc = 'A'; /* close enough */
- } else if (c == '\n') {
- if (prevc == ' ' || prevc == '\t') {
- putc('=', outfile); /* soft & hard lines */
- putc(c, outfile);
- }
- putc(c, outfile);
- ct = 0;
- prevc = c;
- } else {
- if (c == 'F' && prevc == '\n') {
- /* HORRIBLE but clever hack suggested by MTR for sendmail-avoidance */
- c = getc(infile);
- if (c == 'r') {
- c = getc(infile);
- if (c == 'o') {
- c = getc(infile);
- if (c == 'm') {
- c = getc(infile);
- if (c == ' ') {
- /* This is the case we are looking for */
- fputs("=46rom", outfile);
- ct += 6;
- } else {
- fputs("From", outfile);
- ct += 4;
- }
- } else {
- fputs("Fro", outfile);
- ct += 3;
- }
- } else {
- fputs("Fr", outfile);
- ct += 2;
- }
- } else {
- putc('F', outfile);
- ++ct;
- }
- ungetc(c, infile);
- prevc = 'x'; /* close enough -- printable */
- } else { /* END horrible hack */
- putc(c, outfile);
- ++ct;
- prevc = c;
- }
- }
- if (ct > 72) {
- putc('=', outfile);
- putc('\n', outfile);
- ct = 0;
- prevc = '\n';
- }
- }
- if (ct) {
- putc('=', outfile);
- putc('\n', outfile);
- }
+ int c, ct = 0, prevc = 255;
+
+ while ((c = getc(infile)) != EOF) {
+ if ((c < 32 && (c != '\n' && c != '\t'))
+ || (c == '=')
+ || (c >= 127)
+ /* Following line is to avoid single periods alone on lines,
+ which messes up some dumb smtp implementations, sigh... */
+ || (ct == 0 && c == '.')) {
+ putc('=', outfile);
+ putc(basis_hex[c >> 4], outfile);
+ putc(basis_hex[c & 0xF], outfile);
+ ct += 3;
+ prevc = 'A'; /* close enough */
+ } else if (c == '\n') {
+ if (prevc == ' ' || prevc == '\t') {
+ putc('=', outfile); /* soft & hard lines */
+ putc(c, outfile);
+ }
+ putc(c, outfile);
+ ct = 0;
+ prevc = c;
+ } else {
+ if (c == 'F' && prevc == '\n') {
+ /* HORRIBLE but clever hack suggested by MTR for sendmail-avoidance */
+ c = getc(infile);
+ if (c == 'r') {
+ c = getc(infile);
+ if (c == 'o') {
+ c = getc(infile);
+ if (c == 'm') {
+ c = getc(infile);
+ if (c == ' ') {
+ /* This is the case we are looking for */
+ fputs("=46rom", outfile);
+ ct += 6;
+ } else {
+ fputs("From", outfile);
+ ct += 4;
+ }
+ } else {
+ fputs("Fro", outfile);
+ ct += 3;
+ }
+ } else {
+ fputs("Fr", outfile);
+ ct += 2;
+ }
+ } else {
+ putc('F', outfile);
+ ++ct;
+ }
+ ungetc(c, infile);
+ prevc = 'x'; /* close enough -- printable */
+ } else { /* END horrible hack */
+ putc(c, outfile);
+ ++ct;
+ prevc = c;
+ }
+ }
+ if (ct > 72) {
+ putc('=', outfile);
+ putc('\n', outfile);
+ ct = 0;
+ prevc = '\n';
+ }
+ }
+ if (ct) {
+ putc('=', outfile);
+ putc('\n', outfile);
+ }
}
-void fromqp(infile, outfile, boundaries, boundaryct)
-FILE *infile, *outfile;
-char **boundaries;
-int *boundaryct;
+static void
+fromqp(FILE *infile, FILE *outfile, char **boundaries, int *boundaryct)
{
- unsigned int c1, c2;
- int sawnewline = 1, neednewline = 0;
- /* The neednewline hack is necessary because the newline leading into
- a multipart boundary is part of the boundary, not the data */
+ int c1, c2;
+ bool sawnewline = true, neednewline = false;
+ /* The neednewline hack is necessary because the newline leading into
+ a multipart boundary is part of the boundary, not the data */
- while ((c1 = getc(infile)) != EOF) {
- if (sawnewline && boundaries && (c1 == '-')) {
- char Buf[200];
- unsigned char *s;
+ while ((c1 = getc(infile)) != EOF) {
+ if (sawnewline && boundaries && c1 == '-') {
+ char Buf[200];
+ unsigned char *s;
- ungetc(c1, infile);
- fgets(Buf, sizeof(Buf), infile);
- if (boundaries
- && (Buf[0] == '-')
- && (Buf[1] == '-')
- && PendingBoundary(Buf, boundaries, boundaryct)) {
- return;
- }
- /* Not a boundary, now we must treat THIS line as q-p, sigh */
- if (neednewline) {
- putc('\n', outfile);
- neednewline = 0;
- }
- for (s=(unsigned char *) Buf; *s; ++s) {
- if (*s == '=') {
- if (!*++s) break;
- if (*s == '\n') {
- /* ignore it */
- sawnewline = 1;
- } else {
- c1 = hexchar(*s);
- if (!*++s) break;
- c2 = hexchar(*s);
- putc(c1<<4 | c2, outfile);
- }
- } else {
- putc(*s, outfile);
- }
- }
- } else {
- if (neednewline) {
- putc('\n', outfile);
- neednewline = 0;
- }
- if (c1 == '=') {
- sawnewline = 0;
- c1 = getc(infile);
- if (c1 == '\n') {
- /* ignore it */
- sawnewline = 1;
- } else {
- c2 = getc(infile);
- c1 = hexchar(c1);
- c2 = hexchar(c2);
- putc(c1<<4 | c2, outfile);
- if (c2 == '\n') sawnewline = 1;
- }
- } else {
- if (c1 == '\n') {
- sawnewline = 1;
- neednewline = 1;
- } else {
- sawnewline = 0;
- putc(c1, outfile);
- }
- }
- }
- }
- if (neednewline) {
- putc('\n', outfile);
- neednewline = 0;
- }
+ ungetc(c1, infile);
+ fgets(Buf, sizeof(Buf), infile);
+ if (boundaries
+ && Buf[0] == '-'
+ && Buf[1] == '-'
+ && PendingBoundary(Buf, boundaries, boundaryct)) {
+ return;
+ }
+ /* Not a boundary, now we must treat THIS line as q-p, sigh */
+ if (neednewline) {
+ putc('\n', outfile);
+ neednewline = false;
+ }
+ for (s = (unsigned char *)Buf; *s; ++s) {
+ if (*s == '=') {
+ if (*++s == 0)
+ break;
+ if (*s == '\n') {
+ /* ignore it */
+ sawnewline = true;
+ } else {
+ c1 = hexchar(*s);
+ if (*++s == 0)
+ break;
+ c2 = hexchar(*s);
+ putc(c1 << 4 | c2, outfile);
+ }
+ } else {
+ putc(*s, outfile);
+ }
+ }
+ } else {
+ if (neednewline) {
+ putc('\n', outfile);
+ neednewline = false;
+ }
+ if (c1 == '=') {
+ sawnewline = false;
+ c1 = getc(infile);
+ if (c1 == '\n') {
+ /* ignore it */
+ sawnewline = true;
+ } else {
+ c2 = getc(infile);
+ c1 = hexchar(c1);
+ c2 = hexchar(c2);
+ putc(c1 << 4 | c2, outfile);
+ if (c2 == '\n')
+ sawnewline = true;
+ }
+ } else {
+ if (c1 == '\n') {
+ sawnewline = true;
+ neednewline = true;
+ } else {
+ sawnewline = false;
+ putc(c1, outfile);
+ }
+ }
+ }
+ }
+ if (neednewline) {
+ putc('\n', outfile);
+ neednewline = false;
+ }
}
-#define QP 2 /* quoted-printable */
-
-int main(argc, argv)
-int argc;
-char **argv;
+int
+main(int argc, char *argv[])
{
- int encode = 1, i;
- FILE *fp = stdin;
- FILE *fpo = stdout;
+ int i;
+ bool encode = true;
+ FILE *fp = stdin;
+ FILE *fpo = stdout;
- for (i=1; i<argc; ++i) {
- if (argv[i][0] == '-') {
- switch (argv[i][1]) {
- case 'o':
- if (++i >= argc) {
- fprintf(stderr, "mimencode: -o requires a file name.\n");
- exit(-1);
- }
- fpo = fopen(argv[i], "w");
- if (!fpo) {
- perror(argv[i]);
- exit(-1);
- }
- break;
- case 'u':
- encode = 0;
- break;
- default:
- fprintf(stderr,
- "Usage: mmencode [-u] [-o outputfile] [file name]\n");
- exit(-1);
- }
- } else {
- fp = fopen(argv[i], "r");
- if (!fp) {
- perror(argv[i]);
- exit(-1);
- }
- }
- }
- if (encode) toqp(fp, fpo); else fromqp(fp, fpo, NULL, 0);
- return(0);
+ for (i = 1; i < argc; ++i) {
+ if (argv[i][0] == '-') {
+ switch (argv[i][1]) {
+ case 'o':
+ if (++i >= argc) {
+ fprintf(stderr, "mimencode: -o requires a file name.\n");
+ exit(EXIT_FAILURE);
+ }
+ fpo = fopen(argv[i], "w");
+ if (fpo == NULL) {
+ perror(argv[i]);
+ exit(EXIT_FAILURE);
+ }
+ break;
+ case 'u':
+ encode = false;
+ break;
+ default:
+ fprintf(stderr,
+ "Usage: mmencode [-u] [-o outputfile] [file name]\n");
+ exit(EXIT_FAILURE);
+ }
+ } else {
+ fp = fopen(argv[i], "r");
+ if (fp == NULL) {
+ perror(argv[i]);
+ exit(EXIT_FAILURE);
+ }
+ }
+ }
+ if (encode)
+ toqp(fp, fpo);
+ else
+ fromqp(fp, fpo, NULL, 0);
+ return (0);
}
-