openslx-ng/ipxe.git/src/net/tcp, branch v0.9.7

[iscsi] Include credentials in iBFT only if used during iSCSI login

2009-02-20T21:41:00+00:00

Avoid passing credentials in the iBFT that were available but not required for login. This works around a problem in the Microsoft iSCSI initiator, which will refuse to initiate sessions if the CHAP password is fewer than 12 characters, even if the target ends up not asking for CHAP authentication.

[http] Send authentication information whenever username is present

2009-02-17T11:56:27+00:00

Send authentication information if the username is present, even if the password is empty.

[xfer] Make consistent assumptions that xfer metadata can never be NULL

2009-02-15T08:44:22+00:00

The documentation in xfer.h and xfer.c does not say that the metadata parameter is optional in calls such as xfer_deliver_iob_meta() and the deliver_iob() method. However, some code in net/ is prepared to accept a NULL pointer, and xfer_deliver_as_iob() passes a NULL pointer directly to the deliver_iob() method. Fix this mess of conflicting assumptions by making everything assume that the metadata parameter is mandatory, and fixing xfer_deliver_as_iob() to pass in a dummy metadata structure (as is already done in xfer_deliver_iob()).

[http] Allow for URI encodings within username and password

2009-02-13T16:26:43+00:00

[http] Add support for HTTP Basic authentication

2009-02-13T15:43:17+00:00

[iobuf] Add iob_disown() and use it where it simplifies code

2009-02-01T20:16:10+00:00

There are many functions that take ownership of the I/O buffer they are passed as a parameter. The caller should not retain a pointer to the I/O buffer. Use iob_disown() to automatically nullify the caller's pointer, e.g.: xfer_deliver_iob ( xfer, iob_disown ( iobuf ) ); This will ensure that iobuf is set to NULL for any code after the call to xfer_deliver_iob(). iob_disown() is currently used only in places where it simplifies the code, by avoiding an extra line explicitly setting the I/O buffer pointer to NULL. It should ideally be used with each call to any function that takes ownership of an I/O buffer. (The SSA optimisations will ensure that use of iob_disown() gets optimised away in cases where the caller makes no further use of the I/O buffer pointer anyway.) If gcc ever introduces an __attribute__((free)), indicating that use of a function argument after a function call should generate a warning, then we should use this to identify all applicable function call sites, and add iob_disown() as necessary.

[i386] Change [u]int32_t to [unsigned] int, rather than [unsigned] long

2008-11-19T19:15:44+00:00

This brings us in to line with Linux definitions, and also simplifies adding x86_64 support since both platforms have 2-byte shorts, 4-byte ints and 8-byte long longs.

[makefile] Add -Wformat-nonliteral as an extra warning category

2008-10-10T17:41:24+00:00

-Wformat-nonliteral is not enabled by -Wall and needs to be explicitly specified. Modified the few files that use nonliteral format strings to work with this new setting in place. Inspired by a patch from Carl Karsten and an identical patch from Rorschach .

[iscsi] Fix LUN parsing in the iSCSI root-path

2008-09-27T22:53:31+00:00

[iscsi] Change default initiator name prefix to "iqn.2000-01.org.etherboot:"

2008-09-19T16:46:07+00:00

The domain etherboot.org was actually registered on 2000-01-09, not 2000-09-01. (To put it another way, it was registered on 1/9/2000 (US date format) rather than 1/9/2000 (sensible date format); this may illuminate the cause of the error.) "iqn.2000-09.org.etherboot:" is still valid as per RFC3720, but may be surprising to users, so change it to something less unexpected. Thanks to the anonymous contributor for pointing this one out.