【golang】 png: invalid format: not a PNG fileとなる

Question

golangでs3にpng画像をアップロードしようとしています。

まず以下のコードでpngファイルを生成します

	file, err := os.Create("sample.png")
	if err != nil {
		ctx.JSON(http.StatusInternalServerError, ErrorDocument("internal sesrver error"))
	}
	defer file.Close()
	//画像データのコピー
	io.Copy(file, ctx.Request.Body)

そして、下記のコードでリサイズしようとしたのですが、「png: invalid format: not a PNG file」とエラー吐き、デコードに失敗してしまいます。
DecodeConfigを出力してみたところ {<nil> 0 0}となります。
pngファイルは確かに存在していますし、それらのコードを書かずにfileを取得して、そのままアップロードすることはできました。これはどういったエラーなのでしょうか

	file, err := os.Open("sample.png")
	if err != nil {
		log.Println("not found image:", err)
	}
	a, err := png.DecodeConfig(file)
	fmt.Println(a)
	if err != nil {
		log.Fatal(err)
	}
	img, err := png.Decode(file)
	if err != nil {
		log.Println("failed to decode image", err)
	}
	file.Close()

	m := resize.Resize(1280, 780, img, resize.Lanczos3)

	err = png.Encode(file, m)
	if err != nil {
		log.Println("failed to encode image:", err)
	}

Answer 1

DecodeConfig と Decode は同じio.Readerに対し双方を実行することはできません。

多くの画像ファイルの構造は以下の２ブロックあります。

• Header
• Content

Decodeには両方の読み込みが必要です。
DecodeConfigにはHeaderだけを読んで画像ファイルのいろんな情報を得られますが、
Header分だけ読み進んでしまうので、そのまま引き続きfileをDecodeに渡してもHeaderが読めないことでエラーになります。つまり、DecodeやDecodeConfigで要求される引数の型io.Readerは読み進める以外のことはしません（一方通行）。

fileオブジェクトはSeekerインターフェースを満たすのでSeekによって巻き戻しをすれば
両方の処理に通すことができます。

追記

画像がでかい可能性があり、Seekableではないストリームから画像を
よみつつメタ情報も欲しい場合は以下の様に書きます。

https://play.golang.org/p/sLdrO9r9MaF

func ImageDecode(r io.Reader) (image.Config, image.Image, error) {
	header := bytes.NewBuffer(nil)
	info, err := png.DecodeConfig(io.TeeReader(r, header))
	if err != nil {
		return image.Config{}, nil, err
	}
	img, err := png.Decode(io.MultiReader(header, r))
	if err != nil {
		return image.Config{}, nil, err
	}
	return info, img, nil
}

• TeeReaderは読まれた内容のコピーを第二引数のWriterに書くという挙動をとります。
• DecodeConfigは数十バイト以下のヘッダしか読みませんのでheaderバッファにはそのコピーが入ります。
• 画像のDecodeにはheaderの内容とまだ読んでいない残りの画像データをつないでReaderにするMultiReaderでラップしたものを読ませます。

これで巻き戻しを使わずに効率よくメタ情報を得つつ、画像のデコードができます。

Comments

[smilax]

すみません。知識不足のためよくわからないのですが
DecodeにはContentとHeaderの読み込みが必要であり、そのためにはseekによって巻き戻りをする必要があるということですよね。
ファイルオブジェクトにseekは提供されていないようなのですが、
o2, er := file.seek(6, 0)
こういうことではないですか？

[nobonobo]

https://golang.org/pkg/os/#File.Seek