99813aa
diff --git a/libselinux/Makefile b/libselinux/Makefile
99813aa
index fd4f0b1..51469bc 100644
99813aa
--- a/libselinux/Makefile
99813aa
+++ b/libselinux/Makefile
99813aa
@@ -1,4 +1,4 @@
99813aa
-SUBDIRS = src include utils man
99813aa
+SUBDIRS = src include utils man golang
99813aa
 
99813aa
 DISABLE_AVC ?= n
99813aa
 DISABLE_SETRANS ?= n
99813aa
diff --git a/libselinux/golang/Makefile b/libselinux/golang/Makefile
99813aa
new file mode 100644
99813aa
index 0000000..e0be66f
99813aa
--- /dev/null
99813aa
+++ b/libselinux/golang/Makefile
99813aa
@@ -0,0 +1,17 @@
99813aa
+# Installation directories.
99813aa
+PREFIX ?= $(DESTDIR)/usr
99813aa
+GODIR ?= $(PREFIX)/share/gocode/src/selinux
99813aa
+
99813aa
+all:
99813aa
+
99813aa
+install: 
99813aa
+	[ -d $(GODIR) ] || mkdir -p $(GODIR)
99813aa
+	install -m 644 selinux.go $(GODIR)
99813aa
+
99813aa
+test:
99813aa
+
99813aa
+clean:
99813aa
+
99813aa
+indent:
99813aa
+
99813aa
+relabel:
99813aa
diff --git a/libselinux/golang/selinux.go b/libselinux/golang/selinux.go
99813aa
new file mode 100644
99813aa
index 0000000..018c955
99813aa
--- /dev/null
99813aa
+++ b/libselinux/golang/selinux.go
99813aa
@@ -0,0 +1,282 @@
99813aa
+package selinux
99813aa
+
99813aa
+/*
99813aa
+ The selinux package is a go bindings to libselinux required to add selinux
99813aa
+ support to docker.
99813aa
+
99813aa
+ Author Dan Walsh <dwalsh@redhat.com>
99813aa
+
99813aa
+ Used some ideas/code from the go-ini packages https://github.com/vaughan0
99813aa
+ By Vaughan Newton
99813aa
+*/
99813aa
+
99813aa
+// #cgo pkg-config: libselinux
99813aa
+// #include <selinux/selinux.h>
99813aa
+// #include <stdlib.h>
99813aa
+import "C"
99813aa
+import (
99813aa
+	"encoding/binary"
99813aa
+	"crypto/rand"
99813aa
+	"unsafe"
99813aa
+	"fmt"
99813aa
+	"bufio"
99813aa
+	"regexp"
99813aa
+	"io"
99813aa
+	"os"
99813aa
+	"strings"
99813aa
+)
99813aa
+
99813aa
+var (
99813aa
+	assignRegex  = regexp.MustCompile(`^([^=]+)=(.*)$`)
99813aa
+	mcs_list = make(map[string]bool)
99813aa
+)
99813aa
+
99813aa
+func Matchpathcon(path string, mode int) (string, error) {
99813aa
+	var con C.security_context_t
99813aa
+	var scon string
99813aa
+	rc, err := C.matchpathcon(C.CString(path),C.mode_t(mode), &con)
99813aa
+	if rc == 0 {
99813aa
+		scon = C.GoString(con)
99813aa
+		C.free(unsafe.Pointer(con))
99813aa
+	}
99813aa
+	return scon, err
99813aa
+}
99813aa
+
99813aa
+func Setfilecon(path,scon string) (int, error) {
99813aa
+        rc, err := C.lsetfilecon(C.CString(path),C.CString(scon))
99813aa
+	return int(rc), err
99813aa
+}
99813aa
+
99813aa
+func Setexeccon(scon string) (int, error) {
99813aa
+	var val *C.char
99813aa
+	if ! Selinux_enabled() {
99813aa
+		return 0, nil
99813aa
+	}
99813aa
+	if scon != "" {
99813aa
+		val = C.CString(scon)
99813aa
+	} else {
99813aa
+		val = nil
99813aa
+	}
99813aa
+        rc, err := C.setexeccon(val)
99813aa
+	return int(rc), err
99813aa
+}
99813aa
+
99813aa
+type Context struct {
99813aa
+	con []string
99813aa
+}
99813aa
+func (c *Context) Set_user(user string) {
99813aa
+	c.con[0]=user
99813aa
+}
99813aa
+func (c *Context) Get_user() string {
99813aa
+	return c.con[0]
99813aa
+}
99813aa
+func (c *Context) Set_role(role string) {
99813aa
+	c.con[1]=role
99813aa
+}
99813aa
+func (c *Context) Get_role() string {
99813aa
+	return c.con[1]
99813aa
+}
99813aa
+func (c *Context) Set_type(setype string) {
99813aa
+	c.con[2]=setype
99813aa
+}
99813aa
+func (c *Context) Get_type() string {
99813aa
+	return c.con[2]
99813aa
+}
99813aa
+func (c *Context) Set_level(mls string) {
99813aa
+	c.con[3]=mls
99813aa
+}
99813aa
+func (c *Context) Get_level() string {
99813aa
+	return c.con[3]
99813aa
+}
99813aa
+func (c *Context) Get() string{
99813aa
+	return strings.Join(c.con,":")
99813aa
+}
99813aa
+func (c *Context) Set(scon string) {
99813aa
+	c.con = strings.SplitN(scon,":",4)
99813aa
+}
99813aa
+func New_context(scon string) Context {
99813aa
+	var con Context
99813aa
+	con.Set(scon)
99813aa
+	return con
99813aa
+}
99813aa
+
99813aa
+func Is_selinux_enabled() bool {
99813aa
+	b := C.is_selinux_enabled()
99813aa
+	if b > 0 {
99813aa
+		return true;
99813aa
+	}
99813aa
+	return false
99813aa
+}
99813aa
+
99813aa
+func Selinux_enabled() bool {
99813aa
+	b := C.is_selinux_enabled()
99813aa
+	if b > 0 {
99813aa
+		return true;
99813aa
+	}
99813aa
+	return false
99813aa
+}
99813aa
+
99813aa
+const (
99813aa
+	Enforcing = 1
99813aa
+	Permissive = 0
99813aa
+	Disabled = -1
99813aa
+)
99813aa
+
99813aa
+func Selinux_getenforce() int {
99813aa
+	return int(C.security_getenforce())
99813aa
+}
99813aa
+
99813aa
+func Selinux_getenforcemode() (int) {
99813aa
+	var enforce C.int
99813aa
+	C.selinux_getenforcemode(&enforce)
99813aa
+	return int(enforce)
99813aa
+}
99813aa
+
99813aa
+func mcs_add(mcs string) {
99813aa
+	mcs_list[mcs] = true
99813aa
+}
99813aa
+
99813aa
+func mcs_delete(mcs string) {
99813aa
+	mcs_list[mcs] = false
99813aa
+}
99813aa
+
99813aa
+func mcs_exists(mcs string) bool {
99813aa
+	return mcs_list[mcs] 
99813aa
+}
99813aa
+
99813aa
+func uniq_mcs(catRange uint32) string {
99813aa
+	var n uint32
99813aa
+	var c1,c2 uint32
99813aa
+	var mcs string
99813aa
+	for ;; {
99813aa
+		binary.Read(rand.Reader, binary.LittleEndian, &n)
99813aa
+		c1 = n % catRange
99813aa
+		binary.Read(rand.Reader, binary.LittleEndian, &n)
99813aa
+		c2 = n % catRange
99813aa
+		if c1 == c2 {
99813aa
+			continue
99813aa
+		} else {
99813aa
+			if c1 > c2 {
99813aa
+				t := c1
99813aa
+				c1 = c2
99813aa
+				c2 = t
99813aa
+			}
99813aa
+		}
99813aa
+		mcs = fmt.Sprintf("s0:c%d,c%d", c1, c2)
99813aa
+		if mcs_exists(mcs) {
99813aa
+			continue
99813aa
+		}
99813aa
+		mcs_add(mcs)
99813aa
+		break
99813aa
+	}
99813aa
+	return mcs
99813aa
+}
99813aa
+func free_context(process_label string) {
99813aa
+	var scon Context
99813aa
+	scon = New_context(process_label)
99813aa
+	mcs_delete(scon.Get_level())
99813aa
+}
99813aa
+
99813aa
+func Get_lxc_contexts() (process_label string, file_label string) {
99813aa
+	var val, key string
99813aa
+	var bufin *bufio.Reader
99813aa
+	if ! Selinux_enabled() {
99813aa
+		return
99813aa
+	}
99813aa
+	lxc_path := C.GoString(C.selinux_lxc_contexts_path())
99813aa
+	file_label = "system_u:object_r:svirt_sandbox_file_t:s0"
99813aa
+	process_label = "system_u:system_r:svirt_lxc_net_t:s0"
99813aa
+
99813aa
+	in, err := os.Open(lxc_path)
99813aa
+	if err != nil {
99813aa
+		goto exit
99813aa
+	}
99813aa
+
99813aa
+	defer in.Close()
99813aa
+	bufin = bufio.NewReader(in)
99813aa
+
99813aa
+	for done := false; !done; {
99813aa
+		var line string
99813aa
+		if line, err = bufin.ReadString('\n'); err != nil {
99813aa
+			if err == io.EOF {
99813aa
+				done = true
99813aa
+			} else {
99813aa
+				goto exit
99813aa
+			}
99813aa
+		}
99813aa
+		line = strings.TrimSpace(line)
99813aa
+		if len(line) == 0 {
99813aa
+			// Skip blank lines
99813aa
+			continue
99813aa
+		}
99813aa
+		if line[0] == ';' || line[0] == '#' {
99813aa
+			// Skip comments
99813aa
+			continue
99813aa
+		}
99813aa
+		if groups := assignRegex.FindStringSubmatch(line); groups != nil {
99813aa
+			key, val = strings.TrimSpace(groups[1]), strings.TrimSpace(groups[2])
99813aa
+			if key == "process" {
99813aa
+				process_label = strings.Trim(val,"\"")
99813aa
+			}
99813aa
+			if key == "file" {
99813aa
+				file_label = strings.Trim(val,"\"")
99813aa
+			}
99813aa
+		}
99813aa
+	}
99813aa
+exit:
99813aa
+	var scon Context
99813aa
+	mcs := uniq_mcs(1024)
99813aa
+	scon = New_context(process_label)
99813aa
+	scon.Set_level(mcs)
99813aa
+	process_label = scon.Get()
99813aa
+	scon = New_context(file_label)
99813aa
+	scon.Set_level(mcs)
99813aa
+	file_label = scon.Get()
99813aa
+	return process_label, file_label
99813aa
+}
99813aa
+
99813aa
+func CopyLevel (src, dest string) (string, error) {
99813aa
+	if ! Selinux_enabled() {
99813aa
+		return "", nil
99813aa
+	}
99813aa
+	if src == "" {
99813aa
+		return "", nil
99813aa
+	}
99813aa
+	rc, err := C.security_check_context(C.CString(src))
99813aa
+	if rc != 0 {
99813aa
+		return "", err
99813aa
+	}
99813aa
+	rc, err = C.security_check_context(C.CString(dest))
99813aa
+	if rc != 0 {
99813aa
+		return "", err
99813aa
+	}
99813aa
+	scon := New_context(src)
99813aa
+	tcon := New_context(dest)
99813aa
+	tcon.Set_level(scon.Get_level())
99813aa
+	return tcon.Get(), nil
99813aa
+}
99813aa
+
99813aa
+func Test() {
99813aa
+	var plabel,flabel string
99813aa
+	if ! Selinux_enabled() {
99813aa
+		return
99813aa
+	}
99813aa
+
99813aa
+	plabel, flabel = Get_lxc_contexts()
99813aa
+	fmt.Println(plabel)
99813aa
+	fmt.Println(flabel)
99813aa
+	free_context(plabel)
99813aa
+	plabel, flabel = Get_lxc_contexts()
99813aa
+	fmt.Println(plabel)
99813aa
+	fmt.Println(flabel)
99813aa
+	free_context(plabel)
99813aa
+	if Selinux_enabled() {
99813aa
+		fmt.Println("Enabled")
99813aa
+	} else {
99813aa
+		fmt.Println("Disabled")
99813aa
+	}
99813aa
+	fmt.Println(Selinux_getenforce())
99813aa
+	fmt.Println(Selinux_getenforcemode())
99813aa
+	flabel,_ = Matchpathcon("/home/dwalsh/.emacs", 0)
99813aa
+	fmt.Println(flabel)
99813aa
+}
5f9e314
diff --git a/libselinux/src/Makefile b/libselinux/src/Makefile
5f9e314
index 02dd829..6dfdb46 100644
5f9e314
--- a/libselinux/src/Makefile
5f9e314
+++ b/libselinux/src/Makefile
5f9e314
@@ -114,7 +114,7 @@ $(LIBA): $(OBJS)
5f9e314
 	$(RANLIB) $@
5f9e314
 
5f9e314
 $(LIBSO): $(LOBJS)
5f9e314
-	$(CC) $(CFLAGS) -shared -o $@ $^ -lpcre -ldl $(LDFLAGS) -L$(LIBDIR) -Wl,-soname,$(LIBSO),-z,defs,-z,relro
5f9e314
+	$(CC) $(CFLAGS) -shared -o $@ $^ -lpcre -llzma -ldl $(LDFLAGS) -L$(LIBDIR) -Wl,-soname,$(LIBSO),-z,defs,-z,relro
5f9e314
 	ln -sf $@ $(TARGET) 
5f9e314
 
5f9e314
 $(LIBPC): $(LIBPC).in ../VERSION
7e1165a
diff --git a/libselinux/src/fsetfilecon.c b/libselinux/src/fsetfilecon.c
7e1165a
index 9963f7a..37f9d74 100644
7e1165a
--- a/libselinux/src/fsetfilecon.c
7e1165a
+++ b/libselinux/src/fsetfilecon.c
7e1165a
@@ -9,8 +9,12 @@
7e1165a
 
7e1165a
 int fsetfilecon_raw(int fd, const security_context_t context)
7e1165a
 {
7e1165a
-	int rc = fsetxattr(fd, XATTR_NAME_SELINUX, context, strlen(context) + 1,
7e1165a
-			 0);
7e1165a
+	int rc;
7e1165a
+	if (! context) {
7e1165a
+		errno=EINVAL;
7e1165a
+		return -1;
7e1165a
+	}
7e1165a
+	rc = fsetxattr(fd, XATTR_NAME_SELINUX, context, strlen(context) + 1, 0);
7e1165a
 	if (rc < 0 && errno == ENOTSUP) {
7e1165a
 		security_context_t ccontext = NULL;
7e1165a
 		int err = errno;
5f9e314
diff --git a/libselinux/src/load_policy.c b/libselinux/src/load_policy.c
5f9e314
index e419f1a..fdeca93 100644
5f9e314
--- a/libselinux/src/load_policy.c
5f9e314
+++ b/libselinux/src/load_policy.c
5f9e314
@@ -16,6 +16,82 @@
5f9e314
 #include <dlfcn.h>
5f9e314
 #include "policy.h"
5f9e314
 #include <limits.h>
5f9e314
+#include <lzma.h>
5f9e314
+
5f9e314
+static char *lzmaread(int fd, size_t *rsize) {
5f9e314
+	int capacity = 64*1024;
5f9e314
+	char *buf = NULL;
5f9e314
+	int tmpsize = 8 * 1024;
5f9e314
+	unsigned char tmp[tmpsize];
5f9e314
+	unsigned char tmp_out[tmpsize];
5f9e314
+	size_t size = 0;
5f9e314
+	lzma_stream strm = LZMA_STREAM_INIT;
5f9e314
+	lzma_action action = LZMA_RUN;
5f9e314
+	lzma_ret ret;
5f9e314
+	
5f9e314
+	FILE *stream = fdopen (fd, "r");
5f9e314
+	if (!stream) {
5f9e314
+		return NULL;
5f9e314
+	}
5f9e314
+	ret = lzma_stream_decoder(&strm, UINT64_MAX,
5f9e314
+				  LZMA_CONCATENATED);
5f9e314
+	
5f9e314
+	strm.avail_in = 0;
5f9e314
+	strm.next_out = tmp_out;
5f9e314
+	strm.avail_out = tmpsize;
5f9e314
+	
5f9e314
+	buf = (char *) malloc (capacity);
5f9e314
+	if (!buf)
5f9e314
+		goto err;
5f9e314
+	
5f9e314
+	while (1) {
5f9e314
+		if (strm.avail_in == 0) {
5f9e314
+			strm.next_in = tmp;
5f9e314
+			strm.avail_in = fread(tmp, 1, tmpsize, stream);
5f9e314
+			
5f9e314
+			if (ferror(stream)) {
5f9e314
+				// POSIX says that fread() sets errno if
5f9e314
+				// an error occurred. ferror() doesn't
5f9e314
+				// touch errno.
5f9e314
+				goto err;
5f9e314
+			}
5f9e314
+			if (feof(stream)) action = LZMA_FINISH;
5f9e314
+		}
5f9e314
+		
5f9e314
+		ret = lzma_code(&strm, action);
5f9e314
+		
5f9e314
+		// Write and check write error before checking decoder error.
5f9e314
+		// This way as much data as possible gets written to output
5f9e314
+		// even if decoder detected an error.
5f9e314
+		if (strm.avail_out == 0 || ret != LZMA_OK) {
5f9e314
+			const size_t num =  tmpsize - strm.avail_out;
5f9e314
+			if (num > capacity) {
5f9e314
+				buf = (char*) realloc (buf, size*2);
5f9e314
+				capacity = size;
5f9e314
+			}
5f9e314
+			memcpy (buf+size, tmp_out, num);
5f9e314
+			capacity -= num;
5f9e314
+			size += num;
5f9e314
+			strm.next_out = tmp_out;
5f9e314
+			strm.avail_out = tmpsize;
5f9e314
+		}
5f9e314
+		if (ret != LZMA_OK) {
5f9e314
+			if (ret == LZMA_STREAM_END) {
5f9e314
+				break;
5f9e314
+			} else {
5f9e314
+				goto err;
5f9e314
+			}
5f9e314
+		}
5f9e314
+	}
5f9e314
+	*rsize = size;
5f9e314
+	
5f9e314
+	goto exit;
5f9e314
+err:
5f9e314
+	free(buf); buf = NULL;
5f9e314
+exit:
5f9e314
+	lzma_end(&strm;;
5f9e314
+	return buf;
5f9e314
+}
5f9e314
 
5f9e314
 int security_load_policy(void *data, size_t len)
5f9e314
 {
5f9e314
@@ -55,7 +131,7 @@ int selinux_mkload_policy(int preservebools)
5f9e314
 	struct stat sb;
5f9e314
 	struct utsname uts;
5f9e314
 	size_t size;
5f9e314
-	void *map, *data;
5f9e314
+	void *map = NULL, *data=NULL;
5f9e314
 	int fd, rc = -1, prot;
5f9e314
 	sepol_policydb_t *policydb;
5f9e314
 	sepol_policy_file_t *pf;
5f9e314
@@ -181,24 +257,28 @@ checkbool:
5f9e314
 		goto dlclose;
5f9e314
 	}
5f9e314
 
5f9e314
-	if (fstat(fd, &sb) < 0) {
5f9e314
-		fprintf(stderr,
5f9e314
-			"SELinux:  Could not stat policy file %s:  %s\n",
5f9e314
-			path, strerror(errno));
5f9e314
-		goto close;
5f9e314
-	}
5f9e314
-
5f9e314
-	prot = PROT_READ;
5f9e314
-	if (setlocaldefs || preservebools)
5f9e314
-		prot |= PROT_WRITE;
5f9e314
+	data = lzmaread(fd,&size);
5f9e314
 
5f9e314
-	size = sb.st_size;
5f9e314
-	data = map = mmap(NULL, size, prot, MAP_PRIVATE, fd, 0);
5f9e314
-	if (map == MAP_FAILED) {
5f9e314
-		fprintf(stderr,
5f9e314
-			"SELinux:  Could not map policy file %s:  %s\n",
5f9e314
+	if (!data) {
5f9e314
+		if (fstat(fd, &sb) < 0) {
5f9e314
+			fprintf(stderr,
5f9e314
+				"SELinux:  Could not stat policy file %s:  %s\n",
5f9e314
 			path, strerror(errno));
5f9e314
-		goto close;
5f9e314
+			goto close;
5f9e314
+		}
5f9e314
+		
5f9e314
+		prot = PROT_READ;
5f9e314
+		if (setlocaldefs || preservebools)
5f9e314
+			prot |= PROT_WRITE;
5f9e314
+		
5f9e314
+		size = sb.st_size;
5f9e314
+		data = map = mmap(NULL, size, prot, MAP_PRIVATE, fd, 0);
5f9e314
+		if (map == MAP_FAILED) {
5f9e314
+			fprintf(stderr,
5f9e314
+				"SELinux:  Could not map policy file %s:  %s\n",
5f9e314
+				path, strerror(errno));
5f9e314
+			goto close;
5f9e314
+		}
5f9e314
 	}
5f9e314
 
5f9e314
 	if (vers > kernvers && usesepol) {
5f9e314
@@ -210,6 +290,8 @@ checkbool:
5f9e314
 			goto unmap;
5f9e314
 		}
5f9e314
 		policy_file_set_mem(pf, data, size);
5f9e314
+		if (!map)
5f9e314
+			free(data);
5f9e314
 		if (policydb_read(policydb, pf)) {
5f9e314
 			policy_file_free(pf);
5f9e314
 			policydb_free(policydb);
5f9e314
@@ -223,7 +305,8 @@ checkbool:
5f9e314
 				path);
5f9e314
 			policy_file_free(pf);
5f9e314
 			policydb_free(policydb);
5f9e314
-			munmap(map, sb.st_size);
5f9e314
+			if (map)
5f9e314
+				munmap(map, sb.st_size);
5f9e314
 			close(fd);
5f9e314
 			vers--;
5f9e314
 			goto search;
5f9e314
@@ -275,7 +358,7 @@ checkbool:
5f9e314
 #endif
5f9e314
 	}
5f9e314
 
5f9e314
-
5f9e314
+	
5f9e314
 	rc = security_load_policy(data, size);
5f9e314
 	
5f9e314
 	if (rc)
5f9e314
@@ -286,7 +369,8 @@ checkbool:
5f9e314
       unmap:
5f9e314
 	if (data != map)
5f9e314
 		free(data);
5f9e314
-	munmap(map, sb.st_size);
5f9e314
+	if (map)
5f9e314
+		munmap(map, sb.st_size);
5f9e314
       close:
5f9e314
 	close(fd);
5f9e314
       dlclose:
e61de3d
diff --git a/libselinux/src/lsetfilecon.c b/libselinux/src/lsetfilecon.c
7e1165a
index fd9bb26..af2d88c 100644
e61de3d
--- a/libselinux/src/lsetfilecon.c
e61de3d
+++ b/libselinux/src/lsetfilecon.c
7e1165a
@@ -9,8 +9,13 @@
e61de3d
 
e61de3d
 int lsetfilecon_raw(const char *path, const security_context_t context)
e61de3d
 {
e61de3d
-	int rc = lsetxattr(path, XATTR_NAME_SELINUX, context, strlen(context) + 1,
7e1165a
-			 0);
e61de3d
+	int rc;
e61de3d
+	if (! context) {
e61de3d
+		errno=EINVAL;
e61de3d
+		return -1;
e61de3d
+	}
e61de3d
+
7e1165a
+	rc = lsetxattr(path, XATTR_NAME_SELINUX, context, strlen(context) + 1, 0);
e61de3d
 	if (rc < 0 && errno == ENOTSUP) {
e61de3d
 		security_context_t ccontext = NULL;
7e1165a
 		int err = errno;
ca9cea7
diff --git a/libselinux/src/matchpathcon.c b/libselinux/src/matchpathcon.c
01a1f70
index 2d7369e..2a00807 100644
ca9cea7
--- a/libselinux/src/matchpathcon.c
ca9cea7
+++ b/libselinux/src/matchpathcon.c
495b754
@@ -2,6 +2,7 @@
974a6e4
 #include <string.h>
974a6e4
 #include <errno.h>
974a6e4
 #include <stdio.h>
974a6e4
+#include <syslog.h>
974a6e4
 #include "selinux_internal.h"
974a6e4
 #include "label_internal.h"
974a6e4
 #include "callbacks.h"
495b754
@@ -62,7 +63,7 @@ static void
974a6e4
 {
974a6e4
 	va_list ap;
974a6e4
 	va_start(ap, fmt);
974a6e4
-	vfprintf(stderr, fmt, ap);
974a6e4
+	vsyslog(LOG_ERR, fmt, ap);
974a6e4
 	va_end(ap);
974a6e4
 }
b5b41bc
 
7e1165a
diff --git a/libselinux/src/setfilecon.c b/libselinux/src/setfilecon.c
7e1165a
index 50cb228..e617039 100644
7e1165a
--- a/libselinux/src/setfilecon.c
7e1165a
+++ b/libselinux/src/setfilecon.c
7e1165a
@@ -9,8 +9,12 @@
7e1165a
 
7e1165a
 int setfilecon_raw(const char *path, const security_context_t context)
7e1165a
 {
7e1165a
-	int rc = setxattr(path, XATTR_NAME_SELINUX, context, strlen(context) + 1,
7e1165a
-			0);
7e1165a
+	int rc;
7e1165a
+	if (! context) {
7e1165a
+		errno=EINVAL;
7e1165a
+		return -1;
7e1165a
+	}
7e1165a
+	rc = setxattr(path, XATTR_NAME_SELINUX, context, strlen(context) + 1, 0);
7e1165a
 	if (rc < 0 && errno == ENOTSUP) {
7e1165a
 		security_context_t ccontext = NULL;
7e1165a
 		int err = errno;