Ticket #16 (closed バグ報告: fixed)

Opened 11 years ago

Last modified 8 years ago

pnmフォーマットの改行解析ミス

Reported by: anonymous Owned by: ddeguchi
Priority: Milestone: 1.3.5
Component: Mist Version:
Keywords: Cc:

Description

コメント行でなく、かつ改行文字として"\r\n"があると、ポインタを1つ進めすぎてしまうようです。実際に実行して確認はしてません。

pnm.h : 183行目から

00183                 else if( s[ 0 ] == '\r' )
00184                 {
00185                     if( s + 1 != e && s[ 1 ] == '\n' )
00186                     {
00187                         s = s + 2;
00188                     }
00189                     s = s + 1;
00190                     flag = true;
00191                 }

これは

00183                 else if( s[ 0 ] == '\r' )
00184                 {
00185                     if( s + 1 != e && s[ 1 ] == '\n' )
00186                     {
00187                         s = s + 2;
00188                     }
                          else
                          {
00189                         s = s + 1;
                          }
00190                     flag = true;
00191                 }

であるべきかと思います。

Change History

comment:1 Changed 11 years ago by ddeguchi

  • Owner changed from somebody to ddeguchi
  • Status changed from new to assigned

報告ありがとうございます. ご指摘いただいたように,改行コードがCRLFのPNM形式の画像では,ヘッダ部分の解析がうまく行われない場合があることを確認しました. [951] でプログラムのバグを修正したので,最新版のソースをリポジトリからダウンロードしてお使いください.

comment:2 Changed 11 years ago by ddeguchi

  • Milestone set to 1.4

comment:3 Changed 10 years ago by ddeguchi

  • Status changed from assigned to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.