Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Type check dimension in make-array #372

Open
wants to merge 1 commit into
base: cl-compatible
Choose a base branch
from

Conversation

Affonso-Gui
Copy link
Member

Because of 5343112, make-array with null argument is causing segmentation fault in cl-compatible.

(make-array nil)
;p=pointer?(0x4d7f3b8)
;; Segmentation Fault.
;; in (slime:slimetop)
;; You are still in a signal handler.
;;Try reset or throw to upper level as soon as possible.
;; code=698178160 x=299d5940 addr=fffe

Until now using nil caused integer expected errors in

EusLisp/lisp/l/array.l

Lines 104 to 106 in 7ee9f2e

(let* ((i 0)
(rank (length dim))
(total-size (apply #'* dim)))

@Affonso-Gui
Copy link
Member Author

Travis jessie tests are skipped as described in #370.

Other tests are successful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant