Patch from agmiklas for NULL termination bug in ACL translation. Than…

…ks\!
jmhodges · Jul 20, 2010 · 93b6936 · 93b6936
1 parent 60249ff
commit 93b6936
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/.gitignore b/.gitignore
@@ -4,3 +4,4 @@
 *~
 *.log
 Makefile
+pkg
diff --git a/ext/zookeeper_lib.c b/ext/zookeeper_lib.c
@@ -435,13 +435,15 @@ struct Id zkrb_ruby_to_id(VALUE rubyid) {
   if (scheme != Qnil) {
     id.scheme = malloc(RSTRING(scheme)->len + 1);
     strncpy(id.scheme, RSTRING(scheme)->ptr, RSTRING(scheme)->len);
+    id.scheme[RSTRING(scheme)->len] = '\0';
   } else {
     id.scheme = NULL;
   }
 
   if (ident != Qnil) {
     id.id     = malloc(RSTRING(ident)->len + 1);
     strncpy(id.id, RSTRING(ident)->ptr, RSTRING(ident)->len);
+    id.id[RSTRING(ident)->len] = '\0';
   } else {
     id.id = NULL;
   }