330701 - NS_NewPipe with default segment size ignores max size

Status: RESOLVED FIXED

(Core :: Networking, defect)

Description

[jhp (no longer active)]

From bug 315767:

Darin: "The problem is that we were passing 0 for the segmentSize of
the pipe, which causes the pipe to use its default configuration, ignoring the
value of maxSize passed to it.  The real fix to this bug is to change
NS_NewPipe to honor maxSize even when segmentSize is 0."

Comment 1

[jhp (no longer active)]

Attachment: patch

Moved the NS_NewPipe impl code out of the IDL file and into nsPipe3.cpp, so it can use the DEFAULT_SEGMENT_SIZE define.  I also special cased maxSize == PR_UINT32_MAX to pass a segmentCount of PR_UINT32_MAX, since in both cases this means "infinite", according to the docs.

Comment 2

[jhp (no longer active)]

Hmm, this patch would break any embedders who are using this API, right?  How much does that matter, considering that it's not frozen?

Comment 3

[Darin Fisher]

No worries there.  The symbol is only exported by libxpcom_core.so, which doesn't even exist in a xulrunner build.

Comment 4

[Darin Fisher]

Comment on attachment 215289 [details] [diff] [review]
patch

Please add this new function to xpcom/build/dlldeps.cpp to avoid breaking the windows build :)

Comment 5

[jhp (no longer active)]

Checked in to trunk.  -> FIXED

Darin, do you think this needs to go on the 1.8 branch?

Comment 6

[Darin Fisher]

No, let's only put it on the 1.8 branch if there is a need.  People can work around this bug.