Add go.mod file

.travis.yml

@@ -2,4 +2,7 @@ sudo:     false
 language: go
 
 go:
-  - 1.4
+  - 1.13.4
+
+os:
+  - linux

crypt.go → crypt_bsd.go

@@ -1,4 +1,4 @@
-// +build darwin freebsd netbsd
+// +build freebsd openbsd
 
 // Package crypt provides wrappers around functions available in crypt.h
 //
@@ -40,7 +40,9 @@ func Crypt(pass, salt string) (string, error) {
 	if c_enc == nil {
 		return "", err
 	}
-	defer C.free(unsafe.Pointer(c_enc))
+	// returned pointer points to static data which is overwritten
+	// in each call, so dont free it
+	// defer C.free(unsafe.Pointer(c_enc))
 
 	// Return nil error if the string is non-nil.
 	// As per the errno.h manpage, functions are allowed to set errno

crypt_r.go → crypt_linux.go

@@ -11,14 +11,13 @@ import (
 	"unsafe"
 )
 
+
 /*
 #cgo LDFLAGS: -lcrypt
-
 #define _GNU_SOURCE
-
-#include <stdlib.h>
-#include <string.h>
-#include <crypt.h>
+#include <stdlib.h> // for NULL
+#include <string.h> // for strlen
+#include <crypt.h>  // for crypt()
 
 char *gnu_ext_crypt(char *pass, char *salt) {
   char *enc = NULL;
@@ -32,7 +31,7 @@ char *gnu_ext_crypt(char *pass, char *salt) {
   }
 
   ret = (char *)malloc(strlen(enc)+1); // for trailing null
-  strncpy(ret, enc, strlen(enc));
+  memcpy(ret, enc, strlen(enc));
   ret[strlen(enc)]= '\0'; // paranoid
 
   return ret;

crypt_test.go

@@ -1,7 +1,6 @@
 package crypt
 
 import (
-	"strings"
 	"testing"
 )
 
@@ -40,13 +39,6 @@ func TestCryptMD5(t *testing.T) {
 }
 
 func TestCryptErrors(t *testing.T) {
-	FixedMinorVersion := "17"
-
-	tokens := strings.Split(LibCVersion(), ".")
-	if tokens[1] < "17" {
-		t.Skipf("Skipping error tests. libc version too old (got: %s.%s, need: 2.%s)",
-			tokens[0], tokens[1], FixedMinorVersion)
-	}
 
 	tests := [][]string{
 		{"no salt", ""},

go.mod

@@ -0,0 +1,3 @@
+module github.com/amoghe/go-crypt
+
+go 1.13