vox-etf 0.1.4 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23e933f7516643febe38cf5e2d8ee5e9f1ab3fe68ff1fb6dd97c6fa62ff9aa4f
-  data.tar.gz: 84aba301392e4726c27d60be188965cd0657cf312a89e839e315b6a02992c023
+  metadata.gz: 07ad7f903c0086a1f59466dad80db4f037d927df1f6cc806f20030ebb8bdf624
+  data.tar.gz: 433e74a263635e5a41c7b06eb1e90ae16713bea01375811e40074db9a3b3ec83
 SHA512:
-  metadata.gz: 8772aad812bc659a49137e275e369c91dc32923f1ebb1dbbd596bdbc5b9d63d325f0d62cc8379e3dc7d050d4ed2e6112aee88437923b6fcef92b963668596af5
-  data.tar.gz: bcf8b1923f65051c018c092d41e797d3dd4760dce007f5e82dd63a2b139214009c658ee96a508c0aedb245c43dde931bb0b09841684f9173f3c9e96f4d9e217c
+  metadata.gz: 87d8bfa9215395b601017a864bb641cc912021be1809c9ee9f13d11a325ac785a974ad7210e2c7fad29b951e0afc44e5592daf4f766a07211cc5ca5c87657f11
+  data.tar.gz: 80bb347731c1379b37b4d5a9bf28d23675cb7c5e71e61df53c6c4af67336546be785215b57ce2aa8599447d661c26f4fe1c7cc951435207857fa7a21a15c67db

data/ext/vox/decoder.hpp

@@ -1,5 +1,6 @@
 #pragma once
 #include <zlib.h>
+#include "./etf.hpp"
 #include "ruby.h"
 #include "erlpack/sysdep.h"
 #include "erlpack/constants.h"
@@ -360,11 +361,12 @@ namespace etf
 
         VALUE decode_compressed()
         {
+#if HAVE_ZLIB_H
             const uint32_t decompressed_size = read32();
 
-            uint64_t source_size = decompressed_size;
+            unsigned long source_size = decompressed_size;
             uint8_t *out_buffer = (uint8_t *)malloc(decompressed_size);
-            const int ret = uncompress(out_buffer, &source_size, (const unsigned char *)(data + offset), (uint64_t)(size - offset));
+            const int ret = uncompress(out_buffer, &source_size, (const unsigned char *)(data + offset), (uLong)(size - offset));
 
             offset += source_size;
             if (ret != Z_OK)
@@ -378,6 +380,10 @@ namespace etf
             VALUE value = decompressed.decode_term();
             free(out_buffer);
             return value;
+#else
+            rb_raise(rb_eArgError, "vox-etf was compiled without zlib support can cannot decode the compressed term.");
+            return Qnil;
+#endif
         }
     };
 } // namespace etf

data/ext/vox/encoder.hpp

@@ -1,6 +1,7 @@
 #include "erlpack/encoder.h"
 #include "erlpack/constants.h"
-#include "etf.hpp"
+#include "./etf.hpp"
+#include "ruby.h"
 
 namespace etf
 {
@@ -98,11 +99,11 @@ namespace etf
 
         void encode_fixnum(VALUE fixnum)
         {
-            long l = FIX2LONG(fixnum);
-            if (l > 0 && l <= 0xFF)
-                erlpack_append_small_integer(erl_buff, (unsigned char)l);
+            uint32_t n = NUM2UINT(fixnum);
+            if (n > 0 && n <= UINT8_MAX)
+                erlpack_append_small_integer(erl_buff, (uint8_t)n);
             else
-                erlpack_append_integer(erl_buff, l);
+                erlpack_append_integer(erl_buff, n);
         }
 
         void encode_bignum(VALUE bignum)
@@ -137,15 +138,20 @@ namespace etf
 
         void encode_array(VALUE array)
         {
-            uint32_t size = RARRAY_LEN(array);
+            uint64_t size = RARRAY_LEN(array);
             if (size == 0)
             {
                 erlpack_append_nil_ext(erl_buff);
                 return;
             }
+            else if (size > UINT32_MAX)
+            {
+                rb_raise(rb_eRangeError, "Array size is too large to fit into a 32 bit integer.");
+                return;
+            }
 
             erlpack_append_list_header(erl_buff, size);
-            for (uint32_t index = 0; index < size; index++)
+            for (size_t index = 0; index < size; index++)
             {
                 encode_object(RARRAY_AREF(array, index));
             }
@@ -155,8 +161,7 @@ namespace etf
 
         void encode_symbol(VALUE symbol)
         {
-            VALUE str = rb_sym2str(symbol);
-            erlpack_append_atom_utf8(erl_buff, RSTRING_PTR(str), RSTRING_LEN(str));
+            encode_string(rb_sym2str(symbol));
         }
 
         void encode_string(VALUE string)
@@ -166,13 +171,20 @@ namespace etf
 
         void encode_hash(VALUE hash)
         {
-            uint32_t size = RHASH_SIZE(hash);
+            uint64_t size = RHASH_SIZE(hash);
+            if (size > UINT32_MAX)
+            {
+                rb_raise(rb_eRangeError, "Hash size is too large to fit into a 32 bit integer");
+                return;
+            }
+
             erlpack_append_map_header(erl_buff, size);
             VALUE keys = rb_funcall(hash, rb_intern("keys"), 0);
 
             for (uint32_t index = 0; index < size * 2; index += 2)
             {
                 VALUE key = RARRAY_AREF(keys, index / 2);
+                // Normalize keys to strings because discord
                 encode_object(key);
                 encode_object(rb_hash_aref(hash, key));
             }

data/ext/vox/etf.hpp

@@ -1,5 +1,6 @@
 #pragma once
 
+#include "./extconf.h"
 #define ETF_VERSION 131
 
 VALUE decode(VALUE self, VALUE input);

data/ext/vox/extconf.rb

@@ -1,13 +1,16 @@
 # frozen_string_literal: true
 
-require('mkmf-rice')
+require('mkmf')
 
 find_header('ruby.h')
-find_header('zlib.h')
 find_header('stddef.h')
 find_header('stdlib.h')
 find_header('stdint.h')
 find_header('limits.h')
 find_header('string.h')
+have_header('zlib.h')
+have_library('z')
+
+create_header
 
 create_makefile('vox/etf')

data/lib/vox/etf/version.rb

@@ -21,6 +21,6 @@ module Vox
     #   end
 
     # Gem version
-    VERSION = '0.1.4'
+    VERSION = '0.1.9'
   end
 end

data/vox-etf.gemspec

@@ -30,7 +30,6 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
   spec.extensions << 'ext/vox/extconf.rb'
 
-  spec.add_dependency 'rice', '~> 2.2.0'
   spec.add_development_dependency 'rake', '~> 12.3.3'
   spec.add_development_dependency 'rake-compiler', '~> 1.1.1'
   spec.add_development_dependency 'rspec', '~> 3.9.0'

metadata

@@ -1,29 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: vox-etf
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.9
 platform: ruby
 authors:
 - Matthew Carey
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-09-11 00:00:00.000000000 Z
+date: 2020-09-20 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: rice
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.2.0
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.2.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement