Jakarta-Commons/NetのFTPTimestampParserImpl閏年バグ

目次

• 概要
• 詳細
• 原因
• 修正
• そのほか

概要

調べてみると、FTPサーバから拾ったrawのfile entryは閏日日付のファイルも含まれるのですが、戻り値のFTPFileインスタンスを生成する処理中で、サーバの結果の行からFTPFileインスタンスのフィールドに値を刺す箇所でエラーになるため、閏日日付のファイルのエントリがなくなっていることがわかりました。

具体的には、タイムスタンプの部分を切り取って SimpleDateFormat?.parse(String tstr, ParsePosition? pp)に食わせるところで、閏日の場合は java.util.Dateのインスタンスが返らないで、parseエラーを示すnullが返り、その場合の対象行は爽やかにExceptionも出さずになかったことにするという感じになってました。この結果サーバに存在してFTPサーバの応答に含まれるファイルが getFiles()の結果に含まれなくなります。

詳細

原因

// Temporarily add the current year to the short date time
// to cope with short-date leap year strings.
// e.g. Java's DateFormatter will assume that "Feb 29 12:00" refers to 
// Feb 29 1970 (an invalid date) rather than a potentially valid leap year date.
// TODO This is a HORRENDOUS hack. Remove at first opportunity.

なので、このバグが発生するのは2/29に作成されたファイルを同じ年の半年以内にlistした場合(=FTPサーバからの応答では、今年の場合の書式が MMM dd HH:ssになり、現在時刻とのdeltaが半年の場合は MMM dd yyyy）になります。自前でテストしてたときは先週(2008-02-23付近)なので、テストする上で4年前の閏年の日付を使っていたのですが、その場合に本事象は発生せず、気がつきませんでした。 orz

修正

Commons Netの本体を差し替えてbuildしましょう。

そのほか

• 1. バグを追うときに、jdbを使っていたのでその取説でも書こうと思ったんですが、燃え尽きましたｗ 気が向いたら書きます。