59805 - ftp through proxy causes assertion

Status: VERIFIED FIXED

(Core Graveyard :: Networking: FTP, defect, P3)

Description

[Bradley Baetz]

When I use my proxy for an ftp connection, CVS mozilla (as of a few hours ago)
spits out:

###!!! ASSERTION: nsHTTPChannel::SetTransferOffset: 'Not Reached', file
nsHTTPChannel.cpp, line 489
###!!! Break: at file nsHTTPChannel.cpp, line 489
###!!! ASSERTION: nsHTTPChannel::SetTransferCount: 'Not Reached', file
nsHTTPChannel.cpp, line 503
###!!! Break: at file nsHTTPChannel.cpp, line 503

This occurs on any directory or file download. The ftp operation still succeeds,
even with these assertions.

Comment 1

[Darin Fisher]

Reporter: can you verify this bug against a recent build like Netscape 6.0?
Thanks!

Comment 2

[Bradley Baetz]

Yep, still happens in a build which was pulled from CVS about 24 hours ago.

Comment 3

[Bradley Baetz]

Its almost certainly due to lines 330-331 of nsFTPChannel.cpp, which call those
functions on the proxy channel, and then ignore errors for the result:

// rjc says: ignore errors on SetTransferOffset() and
// SetTransferCount() as they may be unimplemented

Comment 4

[Gagan]

ftp bugs to component:ftp

Comment 5

[Bradley Baetz]

This got fixed somewhere along the way, probably when those functions were
removed as part of the necko branch landing.

Comment 6

[Tom Everingham]

verified per bradley's comments