`command return`以降の終了ステータスがゼロ以外の状態であることを確認||

次のコードパッチが見つかりました。

function some_fun() {    
#[...]
    [ -d ${MKAPP_BUILDDIR}/tmp ] || mkdir ${MKAPP_BUILDDIR}/tmp || return 1

    if [ $? -ne 0 ]; then   # never true?
            mount -t tmpfs none ${MKAPP_BUILDDIR}/tmp || return 1
    fi
}

私が理解したところによれば、tmpビルドディレクトリにディレクトリがあることを確認し、そうでない場合はディレクトリを作成しようとします。もし失敗すると、関数はステータスコード1で終了します。

もしそうなら、以下のifステートメントで失敗した終了コードをチェックするのは無意味ですか？それ以前は、失敗したコマンドはありません。関数が返され、if文が処理されなくなるためです。

私が正しく考えているのか、それとも構文が私をだましているのでしょうか？

ところで、私はbashを使用しています。