引数を指定しないで実行するとsegmentation faultする

yoshikaw commented 10 years ago

いつのバージョンからか分かりませんが、引数を指定しないでimg2sixelを実行するとsegmentation faultします。(0.19.1で確認)

ファイルを指定せずに起動するケースはほとんど無いと思いますが、引数に何も指定しない場合の読み込むファイルのところで落ちているような気がします。

% uname -v
Darwin Kernel Version 13.2.0: Thu Apr 17 23:03:13 PDT 2014; root:xnu-2422.100.13~1/RELEASE_X86_64

% img2sixel
zsh: segmentation fault  img2sixel

% uname -v
FreeBSD 10.0-RELEASE-p3 #0: Tue May 13 18:31:10 UTC 2014     root@amd64-builder.daemonology.net:/usr/obj/usr/src/sys/GENERIC

% img2sixel
zsh: segmentation fault (core dumped)  img2sixel

% gdb img2sixel ./img2sixel.core
[...]
#0  0x00000008054582b7 in strstr () from /lib/libc.so.7
[New Thread 806806400 (LWP 100662/img2sixel)]
(gdb) backtrace
#0  0x00000008054582b7 in strstr () from /lib/libc.so.7
#1  0x0000000000419294 in load_image_file (filename=0x0, psx=0x7fffffffd500, psy=0x7fffffffd4fc) at loader.c:247
#2  0x0000000000402d7f in convert_to_sixel (filename=0x41c351 "//", reqcolors=Cannot access memory at address 0x100
) at img2sixel.c:130
#3  0x0000000000402b78 in main (argc=<value optimized out>, argv=0x7fffffffd860) at img2sixel.c:503

saitoha commented 10 years ago

ありがとうございます。 ./configure --with-curl でビルドしたとき、strstrにNULLが渡って再現します。これから直します。

yoshikaw commented 10 years ago

お手数をおかけします。

gitのcommitやconifugreオプションについて記載するのを忘れておりました。

saitoha commented 10 years ago

2e51c6a051bd35134bf5caa5f3e364115d730e4b / v0.19.3 でmasterに入りました。

yoshikaw commented 10 years ago

確認しました。すばやいご対応ありがとうございます。

saitoha / libsixel

引数を指定しないで実行するとsegmentation faultする #2