[Fix] Stackoverflow in BlockSparseArray constructor with wrong number of axes (#109)

Author: lkdvos

Project.toml

@@ -1,7 +1,7 @@
 name = "BlockSparseArrays"
 uuid = "2c9a651f-6452-4ace-a6ac-809f4280fbb4"
 authors = ["ITensor developers <[email protected]> and contributors"]
-version = "0.4.5"
+version = "0.4.6"
 
 [deps]
 Adapt = "79e6a3ab-5dfb-504d-930d-738a2a938a0e"

src/blocksparsearray/blocksparsearray.jl

@@ -157,6 +157,8 @@ function BlockSparseArray{T,N,A}(
   ::UndefInitializer,
   dims::Tuple{AbstractVector{<:Integer},Vararg{AbstractVector{<:Integer}}},
 ) where {T,N,A<:AbstractArray{T,N}}
+  length(dims) == N ||
+    throw(ArgumentError("Length of dims doesn't match number of dimensions."))
   return BlockSparseArray{T,N,A}(undef, blockedrange.(dims))
 end
 

test/test_basics.jl

@@ -97,6 +97,7 @@ arrayts = (Array, JLArray)
         @test iszero(blockstoredlength(a))
         @test iszero(storedlength(a))
       end
+      T != BlockSparseArray{elt} && @test_throws ArgumentError T(undef, bs[1:1])
     end
 
     # BlockSparseVector