capng_c 0.1.3 → 0.1.8

data/ext/capng/capng.h

@@ -19,29 +19,38 @@
 #include <ruby/io.h>
 
 #include <cap-ng.h>
+#include <fcntl.h>
 #include <stdio.h>
-#include <sys/types.h>
 #include <sys/stat.h>
-#include <fcntl.h>
+#include <sys/types.h>
 
-VALUE rb_cCapNG;
-VALUE rb_cCapNGPrint;
-VALUE rb_cCapability;
-VALUE rb_cState;
-VALUE rb_mAction;
-VALUE rb_mSelect;
-VALUE rb_mType;
-VALUE rb_mResult;
-VALUE rb_mPrint;
-VALUE rb_mFlags;
+extern VALUE rb_cCapNG;
+extern VALUE rb_cCapNGPrint;
+extern VALUE rb_cCapability;
+extern VALUE rb_cState;
+extern VALUE rb_mAction;
+extern VALUE rb_mSelect;
+extern VALUE rb_mType;
+extern VALUE rb_mResult;
+extern VALUE rb_mPrint;
+extern VALUE rb_mFlags;
 
-capng_select_t select_name_to_select_type(char *select_name);
-capng_act_t action_name_to_action_type(char *action_name);
-capng_print_t print_name_to_print_type(char *print_name);
-capng_type_t capability_type_name_to_capability_type(char *capability_name);
+capng_select_t
+select_name_to_select_type(char* select_name);
+capng_act_t
+action_name_to_action_type(char* action_name);
+capng_print_t
+print_name_to_print_type(char* print_name);
+capng_type_t
+capability_type_name_to_capability_type(char* capability_name);
 
 void Init_capng_capability(VALUE);
 void Init_capng_enum(VALUE);
+void Init_capng_enum_action(VALUE);
+void Init_capng_enum_flags(VALUE);
+void Init_capng_enum_result(VALUE);
+void Init_capng_enum_select(VALUE);
+void Init_capng_enum_type(VALUE);
 void Init_capng_print(VALUE);
 void Init_capng_state(VALUE);
 #endif // _CAPNG_H

data/ext/capng/enum-action.c

@@ -0,0 +1,35 @@
+/* capng_c */
+/* Copyright 2020- Hiroshi Hatake*/
+/* */
+/* Licensed under the Apache License, Version 2.0 (the "License"); */
+/* you may not use this file except in compliance with the License. */
+/* You may obtain a copy of the License at */
+/*     http://www.apache.org/licenses/LICENSE-2.0 */
+/* Unless required by applicable law or agreed to in writing, software */
+/* distributed under the License is distributed on an "AS IS" BASIS, */
+/* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. */
+/* See the License for the specific language governing permissions and */
+/* limitations under the License. */
+
+/* clang-format off */
+/*
+ * Document-module: CapNG::Action
+ *
+ * Define ADD or Drop constants.
+ *
+ */
+/* clang-format on */
+
+#include <capng.h>
+
+void
+Init_capng_enum_action(VALUE rb_cCapNG)
+{
+  VALUE rb_mAction = rb_define_module_under(rb_cCapNG, "Action");
+
+  // capng_cat_t enum constants
+  /* Mark as DROP action */
+  rb_define_const(rb_mAction, "DROP", INT2NUM(CAPNG_DROP));
+  /* Mark as ADD action */
+  rb_define_const(rb_mAction, "ADD", INT2NUM(CAPNG_ADD));
+}

data/ext/capng/enum-flags.c

@@ -0,0 +1,44 @@
+/* capng_c */
+/* Copyright 2020- Hiroshi Hatake*/
+/* */
+/* Licensed under the Apache License, Version 2.0 (the "License"); */
+/* you may not use this file except in compliance with the License. */
+/* You may obtain a copy of the License at */
+/*     http://www.apache.org/licenses/LICENSE-2.0 */
+/* Unless required by applicable law or agreed to in writing, software */
+/* distributed under the License is distributed on an "AS IS" BASIS, */
+/* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. */
+/* See the License for the specific language governing permissions and */
+/* limitations under the License. */
+
+/* clang-format off */
+/*
+ * Document-module: CapNG::Flags
+ *
+ * Define NO_FLAGS, DROP_SUPP_GRP, CLEAR_BOUNDING, and INIT_SUPP_GRP (if available) constants.
+ */
+/* clang-format on */
+
+#include <capng.h>
+
+void
+Init_capng_enum_flags(VALUE rb_cCapNG)
+{
+  VALUE rb_mFlags = rb_define_module_under(rb_cCapNG, "Flags");
+
+  // capng_flags_t enum constants
+  /* Simply change uid and retain specified capabilities and that's
+   * all. */
+  rb_define_const(rb_mFlags, "NO_FLAG", LONG2NUM(CAPNG_NO_FLAG));
+  /* it will have no effect. */
+  rb_define_const(rb_mFlags, "DROP_SUPP_GRP", LONG2NUM(CAPNG_DROP_SUPP_GRP));
+  /* Clear the bounding set regardless to the internal representation
+   * already setup prior to changing the uid/gid.*/
+  rb_define_const(rb_mFlags, "CLEAR_BOUNDING", LONG2NUM(CAPNG_CLEAR_BOUNDING));
+#if defined(HAVE_CONST_CAPNG_INIT_SUPP_GRP)
+  /* After changing id, initialize any supplement groups that may come with the new
+   * account. If given with Note: Ubuntu Trusty's libcap-ng-dev doesn't have
+   * CAPNG_INIT_SUPP_GRP constant. */
+  rb_define_const(rb_mFlags, "INIT_SUPP_GRP", LONG2NUM(CAPNG_INIT_SUPP_GRP));
+#endif
+}

data/ext/capng/enum-result.c

@@ -0,0 +1,38 @@
+/* capng_c */
+/* Copyright 2020- Hiroshi Hatake*/
+/* */
+/* Licensed under the Apache License, Version 2.0 (the "License"); */
+/* you may not use this file except in compliance with the License. */
+/* You may obtain a copy of the License at */
+/*     http://www.apache.org/licenses/LICENSE-2.0 */
+/* Unless required by applicable law or agreed to in writing, software */
+/* distributed under the License is distributed on an "AS IS" BASIS, */
+/* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. */
+/* See the License for the specific language governing permissions and */
+/* limitations under the License. */
+
+/* clang-format off */
+/*
+ * Document-module: CapNG::Result
+ *
+ * Define FAIL, NONE, PARTAIL, and FULL constants.
+ */
+/* clang-format on */
+
+#include <capng.h>
+
+void
+Init_capng_enum_result(VALUE rb_cCapNG)
+{
+  VALUE rb_mResult = rb_define_module_under(rb_cCapNG, "Result");
+
+  // capng_result_t enum constants
+  /* Fail to retrieve result */
+  rb_define_const(rb_mResult, "FAIL", LONG2NUM(CAPNG_FAIL));
+  /* Retrieve no result */
+  rb_define_const(rb_mResult, "NONE", LONG2NUM(CAPNG_NONE));
+  /* Retrieve partial result. Some capabilities are set up. */
+  rb_define_const(rb_mResult, "PARTIAL", LONG2NUM(CAPNG_PARTIAL));
+  /* Retrieve partial result. Full capabilities are set up. */
+  rb_define_const(rb_mResult, "FULL", LONG2NUM(CAPNG_FULL));
+}

data/ext/capng/enum-select.c

@@ -0,0 +1,39 @@
+/* capng_c */
+/* Copyright 2020- Hiroshi Hatake*/
+/* */
+/* Licensed under the Apache License, Version 2.0 (the "License"); */
+/* you may not use this file except in compliance with the License. */
+/* You may obtain a copy of the License at */
+/*     http://www.apache.org/licenses/LICENSE-2.0 */
+/* Unless required by applicable law or agreed to in writing, software */
+/* distributed under the License is distributed on an "AS IS" BASIS, */
+/* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. */
+/* See the License for the specific language governing permissions and */
+/* limitations under the License. */
+
+/* clang-format off */
+/*
+ * Document-module: CapNG::Select
+ *
+ * Define CAPS, BOUNDS, BOTH, and AMBIENT (if available) constants.
+ */
+/* clang-format on */
+
+#include <capng.h>
+
+void
+Init_capng_enum_select(VALUE rb_cCapNG)
+{
+  VALUE rb_mSelect = rb_define_module_under(rb_cCapNG, "Select");
+
+  // capng_select_t enum constants
+  rb_define_const(rb_mSelect, "CAPS", INT2NUM(CAPNG_SELECT_CAPS));
+  rb_define_const(rb_mSelect, "BOUNDS", INT2NUM(CAPNG_SELECT_BOUNDS));
+  rb_define_const(rb_mSelect, "BOTH", INT2NUM(CAPNG_SELECT_BOTH));
+#if defined(HAVE_CONST_CAPNG_SELECT_AMBIENT)
+  rb_define_const(rb_mSelect, "AMBIENT", INT2NUM(CAPNG_SELECT_AMBIENT));
+#endif
+#if defined(HAVE_CONST_CAPNG_SELECT_ALL)
+  rb_define_const(rb_mSelect, "ALL", INT2NUM(CAPNG_SELECT_ALL));
+#endif
+}

data/ext/capng/enum-type.c

@@ -0,0 +1,42 @@
+/* capng_c */
+/* Copyright 2020- Hiroshi Hatake*/
+/* */
+/* Licensed under the Apache License, Version 2.0 (the "License"); */
+/* you may not use this file except in compliance with the License. */
+/* You may obtain a copy of the License at */
+/*     http://www.apache.org/licenses/LICENSE-2.0 */
+/* Unless required by applicable law or agreed to in writing, software */
+/* distributed under the License is distributed on an "AS IS" BASIS, */
+/* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. */
+/* See the License for the specific language governing permissions and */
+/* limitations under the License. */
+
+/* clang-format off */
+/*
+ * Document-module: CapNG::Type
+ *
+ * Define EFFECTIVE, PERMITTED, INHERITABLE, BOUNDING_SET and AMBIENT (if available) constants.
+ */
+/* clang-format on */
+
+#include <capng.h>
+
+void
+Init_capng_enum_type(VALUE rb_cCapNG)
+{
+  VALUE rb_mType = rb_define_module_under(rb_cCapNG, "Type");
+
+  // capng_type_t enum constants
+  /* Effective type */
+  rb_define_const(rb_mType, "EFFECTIVE", INT2NUM(CAPNG_EFFECTIVE));
+  /* Permitted type */
+  rb_define_const(rb_mType, "PERMITTED", INT2NUM(CAPNG_PERMITTED));
+  /* Inheritable type */
+  rb_define_const(rb_mType, "INHERITABLE", INT2NUM(CAPNG_INHERITABLE));
+  /* Bounding Set type */
+  rb_define_const(rb_mType, "BOUNDING_SET", INT2NUM(CAPNG_BOUNDING_SET));
+#if defined(HAVE_CONST_CAPNG_AMBIENT)
+  /* Ambient type */
+  rb_define_const(rb_mType, "AMBIENT", INT2NUM(CAPNG_AMBIENT));
+#endif
+}

data/ext/capng/enum.c

@@ -13,47 +13,12 @@
 
 #include <capng.h>
 
-void Init_capng_enum(VALUE rb_cCapNG)
+void
+Init_capng_enum(VALUE rb_cCapNG)
 {
-  rb_mAction = rb_define_module_under(rb_cCapNG, "Action");
-  rb_mSelect = rb_define_module_under(rb_cCapNG, "Select");
-  rb_mType = rb_define_module_under(rb_cCapNG, "Type");
-  rb_mResult = rb_define_module_under(rb_cCapNG, "Result");
-  rb_mFlags = rb_define_module_under(rb_cCapNG, "Flags");
-
-  // capng_cat_t enum constants
-  rb_define_const(rb_mAction, "DROP", INT2NUM(CAPNG_DROP));
-  rb_define_const(rb_mAction, "ADD", INT2NUM(CAPNG_ADD));
-
-  // capng_select_t enum constants
-  rb_define_const(rb_mSelect, "CAPS", INT2NUM(CAPNG_SELECT_CAPS));
-  rb_define_const(rb_mSelect, "BOUNDS", INT2NUM(CAPNG_SELECT_BOUNDS));
-  rb_define_const(rb_mSelect, "BOTH", INT2NUM(CAPNG_SELECT_BOTH));
-#if defined(CAPNG_SELECT_AMBIENT)
-  rb_define_const(rb_mSelect, "AMBIENT", INT2NUM(CAPNG_SELECT_AMBIENT));
-#endif
-#if defined(CAPNG_SELECT_ALL)
-  rb_define_const(rb_mSelect, "ALL", INT2NUM(CAPNG_SELECT_ALL));
-#endif
-
-  // capng_type_t enum constants
-  rb_define_const(rb_mType, "EFFECTIVE", INT2NUM(CAPNG_EFFECTIVE));
-  rb_define_const(rb_mType, "PERMITTED", INT2NUM(CAPNG_PERMITTED));
-  rb_define_const(rb_mType, "INHERITABLE", INT2NUM(CAPNG_INHERITABLE));
-  rb_define_const(rb_mType, "BOUNDING_SET", INT2NUM(CAPNG_BOUNDING_SET));
-#if defined(CAPNG_AMBIENT)
-  rb_define_const(rb_mType, "AMBIENT", INT2NUM(CAPNG_AMBIENT));
-#endif
-
-  // capng_result_t enum constants
-  rb_define_const(rb_mResult, "FAIL", LONG2NUM(CAPNG_FAIL));
-  rb_define_const(rb_mResult, "NONE", LONG2NUM(CAPNG_NONE));
-  rb_define_const(rb_mResult, "PARTIAL", LONG2NUM(CAPNG_PARTIAL));
-  rb_define_const(rb_mResult, "FULL", LONG2NUM(CAPNG_FULL));
-
-  // capng_flags_t enum constants
-  rb_define_const(rb_mFlags, "NO_FLAG", LONG2NUM(CAPNG_NO_FLAG));
-  rb_define_const(rb_mFlags, "DROP_SUPP_GRP", LONG2NUM(CAPNG_DROP_SUPP_GRP));
-  rb_define_const(rb_mFlags, "CLEAR_BOUNDING", LONG2NUM(CAPNG_CLEAR_BOUNDING));
-  rb_define_const(rb_mFlags, "INIT_SUPP_GRP", LONG2NUM(CAPNG_INIT_SUPP_GRP));
+  Init_capng_enum_action(rb_cCapNG);
+  Init_capng_enum_flags(rb_cCapNG);
+  Init_capng_enum_result(rb_cCapNG);
+  Init_capng_enum_select(rb_cCapNG);
+  Init_capng_enum_type(rb_cCapNG);
 }

data/ext/capng/extconf.rb

@@ -24,6 +24,10 @@ pkg_config("libcap-ng")
 $CFLAGS << " -Wall -std=c99 -fPIC "
 # $CFLAGS << " -g -O0"
 
+have_const("CAPNG_SELECT_AMBIENT", "cap-ng.h")
+have_const("CAPNG_SELECT_ALL", "cap-ng.h")
+have_const("CAPNG_AMBIENT", "cap-ng.h")
+have_const("CAPNG_INIT_SUPP_GRP", "cap-ng.h")
 have_func("rb_sym2str", "ruby.h")
 have_func("capng_get_caps_fd", "cap-ng.h")
 create_makefile("capng/capng")

data/ext/capng/print.c

@@ -11,23 +11,37 @@
 /* See the License for the specific language governing permissions and */
 /* limitations under the License. */
 
-#include <capng.h>
+/* clang-format off */
+/*
+ * Document-class: CapNG::Print
+ *
+ * Print Linux capabitlities.
+ *
+ * @example
+ *  require 'capng'
+ *
+ *  @print = CapNG::Print.new
+ *  @print.caps_text(:buffer, :effective)
+ */
+/* clang-format on */
 
-struct CapNGPrint {};
+#include <capng.h>
 
-static void capng_print_free(void* capng);
+struct CapNGPrint
+{};
 
-static const rb_data_type_t rb_capng_print_type = {
-  "capng/print",
-  {
-    0,
-    capng_print_free,
-    0,
-  },
-  NULL,
-  NULL,
-  RUBY_TYPED_FREE_IMMEDIATELY
-};
+static void
+capng_print_free(void* capng);
+
+static const rb_data_type_t rb_capng_print_type = { "capng/print",
+                                                    {
+                                                      0,
+                                                      capng_print_free,
+                                                      0,
+                                                    },
+                                                    NULL,
+                                                    NULL,
+                                                    RUBY_TYPED_FREE_IMMEDIATELY };
 
 static void
 capng_print_free(void* ptr)
@@ -40,58 +54,79 @@ rb_capng_print_alloc(VALUE klass)
 {
   VALUE obj;
   struct CapNGPrint* capng_print;
-  obj = TypedData_Make_Struct(
-    klass, struct CapNGPrint, &rb_capng_print_type, capng_print);
+  obj =
+    TypedData_Make_Struct(klass, struct CapNGPrint, &rb_capng_print_type, capng_print);
   return obj;
 }
 
+/*
+ * Initalize Print class.
+ *
+ * @return [nil]
+ *
+ */
 static VALUE
 rb_capng_print_initialize(VALUE self)
 {
   return Qnil;
 }
 
+/*
+ * Print capability as text.
+ *
+ * @param rb_where_name_or_type [String or Symbol or Fixnum] Print target.
+ * @param rb_capability_name_or_type [String or Symbol or Fixnum] Capability name or
+ * constants
+ * @return [Integer]
+ *
+ */
 static VALUE
-rb_capng_print_caps_text(VALUE self, VALUE rb_where_name_or_type, VALUE rb_capability_name_or_type)
+rb_capng_print_caps_text(VALUE self, VALUE rb_where_name_or_type,
+                         VALUE rb_capability_name_or_type)
 {
-  char *result = NULL;
+  char* result = NULL;
   capng_type_t capability_type = 0;
   capng_print_t print_type = 0;
 
   switch (TYPE(rb_capability_name_or_type)) {
-  case T_SYMBOL:
-    capability_type = capability_type_name_to_capability_type(RSTRING_PTR(rb_sym2str(rb_capability_name_or_type)));
-    break;
-  case T_STRING:
-    capability_type = capability_type_name_to_capability_type(StringValuePtr(rb_capability_name_or_type));
-    break;
-  case T_FIXNUM:
-    capability_type = NUM2INT(rb_capability_name_or_type);
-    break;
-  default:
-    rb_raise(rb_eArgError, "Expected a String or a Symbol instance, or a capability type constant");
+    case T_SYMBOL:
+      capability_type = capability_type_name_to_capability_type(
+        RSTRING_PTR(rb_sym2str(rb_capability_name_or_type)));
+      break;
+    case T_STRING:
+      capability_type = capability_type_name_to_capability_type(
+        StringValuePtr(rb_capability_name_or_type));
+      break;
+    case T_FIXNUM:
+      capability_type = NUM2INT(rb_capability_name_or_type);
+      break;
+    default:
+      rb_raise(rb_eArgError,
+               "Expected a String or a Symbol instance, or a capability type constant");
   }
 
   switch (TYPE(rb_where_name_or_type)) {
-  case T_SYMBOL:
-    print_type = print_name_to_print_type(RSTRING_PTR(rb_sym2str(rb_where_name_or_type)));
-    break;
-  case T_STRING:
-    print_type = print_name_to_print_type(StringValuePtr(rb_where_name_or_type));
-    break;
-  case T_FIXNUM:
-    print_type = NUM2INT(rb_where_name_or_type);
-    break;
-  default:
-    rb_raise(rb_eArgError, "Expected a String or a Symbol instance, or a print type constant");
+    case T_SYMBOL:
+      print_type =
+        print_name_to_print_type(RSTRING_PTR(rb_sym2str(rb_where_name_or_type)));
+      break;
+    case T_STRING:
+      print_type = print_name_to_print_type(StringValuePtr(rb_where_name_or_type));
+      break;
+    case T_FIXNUM:
+      print_type = NUM2INT(rb_where_name_or_type);
+      break;
+    default:
+      rb_raise(rb_eArgError,
+               "Expected a String or a Symbol instance, or a print type constant");
   }
 
   switch (print_type) {
-  case CAPNG_PRINT_STDOUT:
-    capng_print_caps_text(CAPNG_PRINT_STDOUT, capability_type);
-    break;
-  case CAPNG_PRINT_BUFFER:
-    result = capng_print_caps_text(CAPNG_PRINT_BUFFER, capability_type);
+    case CAPNG_PRINT_STDOUT:
+      capng_print_caps_text(CAPNG_PRINT_STDOUT, capability_type);
+      break;
+    case CAPNG_PRINT_BUFFER:
+      result = capng_print_caps_text(CAPNG_PRINT_BUFFER, capability_type);
   }
 
   if (result)
@@ -100,47 +135,60 @@ rb_capng_print_caps_text(VALUE self, VALUE rb_where_name_or_type, VALUE rb_capab
     return rb_str_new2("none");
 }
 
+/*
+ * Print capability as numeric.
+ *
+ * @param rb_where_name_or_type [String or Symbol or Fixnum] Print target.
+ * @param rb_select_name_or_enum [String or Symbol or Fixnum] Select set name or constants
+ * @return [Integer]
+ *
+ */
 static VALUE
-rb_capng_print_caps_numeric(VALUE self, VALUE rb_where_name_or_type, VALUE rb_select_name_or_enum)
+rb_capng_print_caps_numeric(VALUE self, VALUE rb_where_name_or_type,
+                            VALUE rb_select_name_or_enum)
 {
-  char *result = NULL;
+  char* result = NULL;
   capng_select_t select = 0;
   capng_print_t print_type = 0;
 
   switch (TYPE(rb_where_name_or_type)) {
-  case T_SYMBOL:
-    print_type = print_name_to_print_type(RSTRING_PTR(rb_sym2str(rb_where_name_or_type)));
-    break;
-  case T_STRING:
-    print_type = print_name_to_print_type(StringValuePtr(rb_where_name_or_type));
-    break;
-  case T_FIXNUM:
-    print_type = NUM2INT(rb_where_name_or_type);
-    break;
-  default:
-    rb_raise(rb_eArgError, "Expected a String or a Symbol instance, or a print type constant");
+    case T_SYMBOL:
+      print_type =
+        print_name_to_print_type(RSTRING_PTR(rb_sym2str(rb_where_name_or_type)));
+      break;
+    case T_STRING:
+      print_type = print_name_to_print_type(StringValuePtr(rb_where_name_or_type));
+      break;
+    case T_FIXNUM:
+      print_type = NUM2INT(rb_where_name_or_type);
+      break;
+    default:
+      rb_raise(rb_eArgError,
+               "Expected a String or a Symbol instance, or a print type constant");
   }
 
   switch (TYPE(rb_select_name_or_enum)) {
-  case T_SYMBOL:
-    select = select_name_to_select_type(RSTRING_PTR(rb_sym2str(rb_select_name_or_enum)));
-    break;
-  case T_STRING:
-    select = select_name_to_select_type(StringValuePtr(rb_select_name_or_enum));
-    break;
-  case T_FIXNUM:
-    select = NUM2INT(rb_select_name_or_enum);
-    break;
-  default:
-    rb_raise(rb_eArgError, "Expected a String or a Symbol instance, or a capability type constant");
+    case T_SYMBOL:
+      select =
+        select_name_to_select_type(RSTRING_PTR(rb_sym2str(rb_select_name_or_enum)));
+      break;
+    case T_STRING:
+      select = select_name_to_select_type(StringValuePtr(rb_select_name_or_enum));
+      break;
+    case T_FIXNUM:
+      select = NUM2INT(rb_select_name_or_enum);
+      break;
+    default:
+      rb_raise(rb_eArgError,
+               "Expected a String or a Symbol instance, or a capability type constant");
   }
 
   switch (print_type) {
-  case CAPNG_PRINT_STDOUT:
-    capng_print_caps_numeric(CAPNG_PRINT_STDOUT, select);
-    break;
-  case CAPNG_PRINT_BUFFER:
-    result = capng_print_caps_numeric(CAPNG_PRINT_BUFFER, select);
+    case CAPNG_PRINT_STDOUT:
+      capng_print_caps_numeric(CAPNG_PRINT_STDOUT, select);
+      break;
+    case CAPNG_PRINT_BUFFER:
+      result = capng_print_caps_numeric(CAPNG_PRINT_BUFFER, select);
   }
 
   if (result)
@@ -149,9 +197,10 @@ rb_capng_print_caps_numeric(VALUE self, VALUE rb_where_name_or_type, VALUE rb_se
     return rb_str_new2("none");
 }
 
-void Init_capng_print(VALUE rb_cCapNG)
+void
+Init_capng_print(VALUE rb_cCapNG)
 {
-  rb_cCapNGPrint = rb_define_class_under(rb_cCapNG, "Print", rb_cObject);
+  VALUE rb_cCapNGPrint = rb_define_class_under(rb_cCapNG, "Print", rb_cObject);
 
   rb_define_alloc_func(rb_cCapNGPrint, rb_capng_print_alloc);
 
@@ -160,6 +209,8 @@ void Init_capng_print(VALUE rb_cCapNG)
   rb_define_method(rb_cCapNGPrint, "caps_numeric", rb_capng_print_caps_numeric, 2);
 
   // capng_print_t enum constants
+  /* Print target into STDOUT. */
   rb_define_const(rb_cCapNGPrint, "STDOUT", LONG2NUM(CAPNG_PRINT_STDOUT));
+  /* Print target into buffer varible. */
   rb_define_const(rb_cCapNGPrint, "BUFFER", LONG2NUM(CAPNG_PRINT_BUFFER));
 }