ruby-libvirt 0.8.3 → 0.8.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cedabfb7ce30484b1ee1781d9c54eecc102f3a02ae4e8bb3939dc76048db072d
-  data.tar.gz: a5bbc0558a140785b26f7687b3d155767f2813baa51271d7114ff7389761755d
+  metadata.gz: 39fda947f631865aa0b6f0583aad469746de1bde687a106f08e9cc40167bb78c
+  data.tar.gz: 010a9b66860d55d01ea1220a896a267151bb0c773261c37076101fb7dcae439e
 SHA512:
-  metadata.gz: 61db9b3ab4593e7712a96bd740aaf2c4ef825605a6773038f679f120e62f08dc9052421fd0b1b67a98e3d34593f93ef90df7f3fdb09cc34174ba2ececd75062f
-  data.tar.gz: 6d8821193b57518149efcda6d2a14c6eadc06895e7388f9b344ae28f56529e4ee70623fcea2fb70683e4541f15e4e72e5baa8071c17bf54cfb2c7273a1174d2e
+  metadata.gz: cc4db18a21728f16e913fd1f23a2b9de39c46e17a9936f72c4cf9d4da7d6f9051a95a55cb750504640e8bc9ba12055d4a505f441c73ccadb32024983dd326e67
+  data.tar.gz: cb461a77f74fe3a69ac5ae4e52ba46a2b2bbaebf759dd4b8301296842c378501e0e3801e506ffd9a0b4d14f560dabe9e7cd3e6c49d77d3b93083b19bff9cdfaa

data/NEWS.rst

@@ -3,6 +3,12 @@ ruby-libvirt releases
 =====================
 
 
+0.8.4 (2024-08-01)
+==================
+
+* Explicitly disallow use of ``new`` for wrapper classes
+
+
 0.8.3 (2024-05-13)
 ==================
 

data/Rakefile

@@ -16,7 +16,7 @@ require 'rdoc/task'
 require 'rubygems/package_task'
 
 PKG_NAME='ruby-libvirt'
-PKG_VERSION='0.8.3'
+PKG_VERSION='0.8.4'
 
 EXT_DIR = "ext/libvirt"
 EXTCONF = "#{EXT_DIR}/extconf.rb"

data/ext/libvirt/common.c

@@ -491,3 +491,14 @@ int ruby_libvirt_get_maxcpus(virConnectPtr conn)
 
     return maxcpu;
 }
+
+/* For classes where Ruby objects are just wrappers around C pointers,
+ * the only acceptable way to create new instances is to use
+ * Connect.create_domain_xml and similar. The use of Domain.new and
+ * friends is explicitly disallowed by providing this functions as
+ * implementation when defining the class
+ */
+VALUE ruby_libvirt_new_not_allowed(int argc, VALUE *argv, VALUE obj)
+{
+    rb_raise(rb_eTypeError, "Not allowed for this class");
+}

data/ext/libvirt/common.h

@@ -268,6 +268,8 @@ struct ruby_libvirt_str_new2_and_ary_store_arg {
 };
 VALUE ruby_libvirt_str_new2_and_ary_store_wrap(VALUE arg);
 
+VALUE ruby_libvirt_new_not_allowed(int argc, VALUE *argv, VALUE obj);
+
 #ifndef RARRAY_LEN
 #define RARRAY_LEN(ar) (RARRAY(ar)->len)
 #endif

data/ext/libvirt/connect.c

@@ -2742,6 +2742,8 @@ void ruby_libvirt_connect_init(void)
 {
     c_connect = rb_define_class_under(m_libvirt, "Connect", rb_cObject);
     rb_undef_alloc_func(c_connect);
+    rb_define_singleton_method(c_connect, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     /*
      * Class Libvirt::Connect::Nodeinfo

data/ext/libvirt/domain.c

@@ -4064,6 +4064,8 @@ void ruby_libvirt_domain_init(void)
 {
     c_domain = rb_define_class_under(m_libvirt, "Domain", rb_cObject);
     rb_undef_alloc_func(c_domain);
+    rb_define_singleton_method(c_domain, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     rb_define_const(c_domain, "NOSTATE", INT2NUM(VIR_DOMAIN_NOSTATE));
     rb_define_const(c_domain, "RUNNING", INT2NUM(VIR_DOMAIN_RUNNING));
@@ -4390,6 +4392,8 @@ void ruby_libvirt_domain_init(void)
      */
     c_domain_snapshot = rb_define_class_under(c_domain, "Snapshot", rb_cObject);
     rb_undef_alloc_func(c_domain_snapshot);
+    rb_define_singleton_method(c_domain_snapshot, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     rb_define_const(c_domain_snapshot, "DELETE_CHILDREN",
                     INT2NUM(VIR_DOMAIN_SNAPSHOT_DELETE_CHILDREN));

data/ext/libvirt/extconf.rb

@@ -1,9 +1,5 @@
 require 'mkmf'
 
-RbConfig::MAKEFILE_CONFIG['CC'] = ENV['CC'] if ENV['CC']
-RbConfig::MAKEFILE_CONFIG['CCDLFLAGS'] = ENV['CFLAGS'] if ENV['CFLAGS']
-RbConfig::MAKEFILE_CONFIG['EXTDLDFLAGS'] = ENV['CFLAGS'] if ENV['CFLAGS']
-
 extension_name = '_libvirt'
 
 unless pkg_config("libvirt")

data/ext/libvirt/interface.c

@@ -175,6 +175,8 @@ void ruby_libvirt_interface_init(void)
 {
     c_interface = rb_define_class_under(m_libvirt, "Interface", rb_cObject);
     rb_undef_alloc_func(c_interface);
+    rb_define_singleton_method(c_interface, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     rb_define_const(c_interface, "XML_INACTIVE",
                     INT2NUM(VIR_INTERFACE_XML_INACTIVE));

data/ext/libvirt/network.c

@@ -330,6 +330,8 @@ void ruby_libvirt_network_init(void)
 {
     c_network = rb_define_class_under(m_libvirt, "Network", rb_cObject);
     rb_undef_alloc_func(c_network);
+    rb_define_singleton_method(c_network, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     rb_define_attr(c_network, "connection", 1, 0);
 

data/ext/libvirt/nodedevice.c

@@ -261,6 +261,8 @@ void ruby_libvirt_nodedevice_init(void)
 {
     c_nodedevice = rb_define_class_under(m_libvirt, "NodeDevice", rb_cObject);
     rb_undef_alloc_func(c_nodedevice);
+    rb_define_singleton_method(c_nodedevice, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     rb_define_attr(c_nodedevice, "connection", 1, 0);
 

data/ext/libvirt/nwfilter.c

@@ -124,6 +124,8 @@ void ruby_libvirt_nwfilter_init(void)
 {
     c_nwfilter = rb_define_class_under(m_libvirt, "NWFilter", rb_cObject);
     rb_undef_alloc_func(c_nwfilter);
+    rb_define_singleton_method(c_nwfilter, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     rb_define_attr(c_nwfilter, "connection", 1, 0);
 

data/ext/libvirt/secret.c

@@ -231,6 +231,8 @@ void ruby_libvirt_secret_init(void)
 {
     c_secret = rb_define_class_under(m_libvirt, "Secret", rb_cObject);
     rb_undef_alloc_func(c_secret);
+    rb_define_singleton_method(c_secret, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     rb_define_attr(c_secret, "connection", 1, 0);
 

data/ext/libvirt/storage.c

@@ -753,6 +753,8 @@ void ruby_libvirt_storage_init(void)
     c_storage_pool = rb_define_class_under(m_libvirt, "StoragePool",
                                            rb_cObject);
     rb_undef_alloc_func(c_storage_pool);
+    rb_define_singleton_method(c_storage_pool, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     rb_define_attr(c_storage_pool, "connection", 1, 0);
 
@@ -866,6 +868,8 @@ void ruby_libvirt_storage_init(void)
     c_storage_vol = rb_define_class_under(m_libvirt, "StorageVol",
                                           rb_cObject);
     rb_undef_alloc_func(c_storage_vol);
+    rb_define_singleton_method(c_storage_vol, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     rb_define_const(c_storage_vol, "XML_INACTIVE",
                     INT2NUM(VIR_STORAGE_XML_INACTIVE));

data/ext/libvirt/stream.c

@@ -361,6 +361,8 @@ void ruby_libvirt_stream_init(void)
 {
     c_stream = rb_define_class_under(m_libvirt, "Stream", rb_cObject);
     rb_undef_alloc_func(c_stream);
+    rb_define_singleton_method(c_stream, "new",
+                               ruby_libvirt_new_not_allowed, -1);
 
     rb_define_attr(c_stream, "connection", 1, 0);
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ruby-libvirt
 version: !ruby/object:Gem::Version
-  version: 0.8.3
+  version: 0.8.4
 platform: ruby
 authors:
 - David Lutterkort
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-05-13 00:00:00.000000000 Z
+date: 2024-08-01 00:00:00.000000000 Z
 dependencies: []
 description: |
   ruby-libvirt allows applications written in Ruby to use the