122913 - Empty FTP does not display properly

Status: VERIFIED FIXED

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

Description

[Gili Tzabari]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:0.9.7+) Gecko/20020122
BuildID:    2002012216



Reproducible: Always
Steps to Reproduce:
1. Visit an empty FTP (that is, the main directory is empty and there is no
parent directory)
2. Mozilla will not update the screen and the FTP information will not show up
3.

Comment 1

[Chris Lyon]

*** Bug 122914 has been marked as a duplicate of this bug. ***

Comment 2

[Adam Hauner]

Do you have any URL?

Comment 3

[Matthias Versen [:Matti]]

confirming with win2k build 20020204..
I used my own server (the 5th server i tried works with mozilla)

Comment 4

[Gili Tzabari]

Here are better repro steps:

1. Open Mozilla
2. Load http://www.google.com/
3. Enter "ftp://[e-mail]/" into URL field and press ENTER
4. Google's website will continue to be displayed even though Mozilla
successfully logs into the FTP

I have a feeling that with the above repro steps you should be able to repro
this under Windows as well.

NOTE: bbs.darktech.org is my DDNS server; it might not be up 100% of the time.

Comment 5

[Matthias Versen [:Matti]]

Attachment: NSPR Log File from my server with the patch from bug 110241

Comment 6

[Bradley Baetz (:bbaetz)]

I can't test your specific server (its down ATM), but this WFM in general on
wu-ftpd on localhost. I'm going to attach a patch which may help. Can someone
who is seeing this on a a current build w/o my patch try it with?

Comment 7

[Bradley Baetz (:bbaetz)]

Attachment: patch to try

If the assertion fires, then this makes a difference

Comment 8

[Matthias Versen [:Matti]]

Tested with an opt build.
This patch fixed the problem with my FTP server and with 
"ftp://[e-mail]/" 

Comment 9

[Syd Logan]

only nsbeta1+ bugs can have milestones, resetting to ---

Comment 10

[Bradley Baetz (:bbaetz)]

nsbeta1 does not apply to non netscape employees (And I have a patch for this!)
Resetting milestone.

I'll attach a patch without the assertion in there. I still can't reproduce it,
but the problem was that because of the DelayedOnStart stuff we were only
sending an OnStop w/o the OnStart if there were no calls to OnDataAvailable.

Comment 11

[Bradley Baetz (:bbaetz)]

Attachment: v2

Comment 12

[Darin Fisher]

Comment on attachment 68224 [details] [diff] [review]
v2

sr=darin

Comment 13

[Doug Turner (:dougt)]

Comment on attachment 68224 [details] [diff] [review]
v2

r=dougt

Comment 14

[Bradley Baetz (:bbaetz)]

Checked in/

Comment 15

[benc]

VERIFIED: RC1, W98.