Trivial fixes

diff --git a/README b/README
index 802056ab261b239612e146fbc06157c6dc9209d8..d16ccebf98188a061c0e3da10448e99085ab5648 100644 (file)
--- a/README
+++ b/README
@@ -1,13 +1,19 @@
 Go high-performance encryption utility.
 
 gohpenc highly resembles hpenc tool (https://github.com/vstakhov/hpenc).
+hpenc solves the problem that there is no simple tool to quickly
+transfer data with encryption and authentication:
 
-Why it was written? hpenc has some problems: it does not work on aarch64
-and sparc64 architectures under FreeBSD (as seen in the port's Makefile)
-and produces incompatible output (unauthenticated after 8192 blocks)
-between FreeBSD and HardenedBSD systems somehow. Instead of painful
-debugging I decided to write something similar on the Go language,
-widening supported platforms.
+* openssl enc -- uses single CPU, no authentication
+* GnuPG -- complex key generation/management, relatively slow
+* OpenSSH -- uses single CPU, not very fast
+
+Why gohpenc was written? hpenc has some problems: it does not work on
+aarch64 and sparc64 architectures under FreeBSD (as seen in the port's
+Makefile) and produces incompatible output (unauthenticated after 8192
+blocks) between FreeBSD and HardenedBSD systems somehow. Instead of
+painful debugging I decided to write something similar on the Go
+language, widening supported platforms.
 
 gohpenc is incompatible with hpenc and much simpler:
 
@@ -27,9 +33,16 @@ But it still satisfies most of hpenc aims:
   dependent libraries contain assembly-optimized code
 * Built-in authentication and integrity check with small data overhead
 
+Usage is very simple:
+
+    $ gohpenc -psk
+    DTGZI5R2HS4YEDSIO56AFKPONE6KJE3Q2QETODDOH3O6UYFPROHQ
+    $ echo "message to be transmitted" | gohpenc -k DTGZI5R2HS4YEDSIO56AFKPONE6KJE3Q2QETODDOH3O6UYFPROHQ > encrypted
+    $ gohpenc -k DTGZI5R2HS4YEDSIO56AFKPONE6KJE3Q2QETODDOH3O6UYFPROHQ -d < encrypted
+
 How encryption/authentication is performed:
 
-* First 32 bytes of the stream contains random data, called salt
+* First 32 bytes of the stream contain random data -- salt
 * BLAKE2X is initialized: unknown length, PSK key as a MAC key. It
   creates XOF that will be used as a KDF
 * Salt is fed into that XOF

diff --git a/src/cypherpunks.ru/gohpenc/base32.go b/src/cypherpunks.ru/gohpenc/base32.go
new file mode 100644 (file)
index 0000000..f84388f
--- /dev/null
+++ b/src/cypherpunks.ru/gohpenc/base32.go
@@ -0,0 +1,23 @@
+package main
+
+import (
+       "encoding/base32"
+       "strings"
+)
+
+func ToBase32(data []byte) string {
+       return strings.TrimRight(base32.StdEncoding.EncodeToString(data), "=")
+}
+
+func FromBase32(data string) ([]byte, error) {
+       padSize := len(data) % 8
+       if padSize != 0 {
+               padSize = 8 - padSize
+               pad := make([]byte, 0, padSize)
+               for i := 0; i < padSize; i++ {
+                       pad = append(pad, '=')
+               }
+               data += string(pad)
+       }
+       return base32.StdEncoding.DecodeString(data)
+}

diff --git a/src/cypherpunks.ru/gohpenc/main.go b/src/cypherpunks.ru/gohpenc/main.go
index 6f44eb12e28ff8ccf88194567df9ca07a73677a3..d4156160c5d60e49be6df327544ebcab3dc1c9ee 100644 (file)
--- a/src/cypherpunks.ru/gohpenc/main.go
+++ b/src/cypherpunks.ru/gohpenc/main.go
@@ -22,14 +22,12 @@ package main
 import (
        "bufio"
        "crypto/rand"
-       "encoding/base32"
        "encoding/binary"
        "flag"
        "fmt"
        "io"
        "os"
        "runtime"
-       "strings"
        "sync"
 
        "golang.org/x/crypto/blake2b"
@@ -41,23 +39,6 @@ const (
        SaltSize = 32
 )
 
-func ToBase32(data []byte) string {
-       return strings.TrimRight(base32.StdEncoding.EncodeToString(data), "=")
-}
-
-func FromBase32(data string) ([]byte, error) {
-       padSize := len(data) % 8
-       if padSize != 0 {
-               padSize = 8 - padSize
-               pad := make([]byte, 0, padSize)
-               for i := 0; i < padSize; i++ {
-                       pad = append(pad, '=')
-               }
-               data += string(pad)
-       }
-       return base32.StdEncoding.DecodeString(data)
-}
-
 type WorkerTask struct {
        key []byte
        n   int
@@ -184,8 +165,7 @@ func main() {
        var w *Worker
        for {
                key := make([]byte, chacha20poly1305.KeySize)
-               _, err = io.ReadFull(keys, key)
-               if err != nil {
+               if _, err = io.ReadFull(keys, key); err != nil {
                        panic(err)
                }
                w = workers[i%len(workers)]
@@ -199,8 +179,7 @@ func main() {
                                panic(err)
                        }
                        n = int(binary.BigEndian.Uint32(w.buf[:LenSize]))
-                       n, err = io.ReadFull(stdin, w.buf[LenSize:LenSize+n+tmpAEAD.Overhead()])
-                       if err != nil {
+                       if n, err = io.ReadFull(stdin, w.buf[LenSize:LenSize+n+tmpAEAD.Overhead()]); err != nil {
                                panic(err)
                        }
                } else {