search for: acceptanyport

..."StationPort" "The port number to which this EFI UDPv4 Protocol instance is bound. If a client of the EFI UDPv4 Protocol does not care about the port number, set StationPort to zero. The EFI UDPv4 Protocol driver will assign a random port number to transmitted UDP packets. Ignored if AcceptAnyPort is set to TRUE." p1385 (EFI_UDP4_PROTOCOL.Transmit()) "SourcePort" "Port from which this packet is sent. It is in host byte order. If this field is set to zero when sending packets, the port that is assigned in EFI_UDP4_PROTOCOL.Configure() is used. If this field is set to zero...

...and acknowledge options, but receive no response at all. Actually, what happen is that before any packet is actually sent (and Transmit is called), GetModeData return a StationPort = 0. Leading to received packets being discarded by the firmware because of the non-matching port and the absence of AcceptAnyPort flag. Local port, thus, can't be saved before the first call to core_udp_send or core_udp_sendto. Two possible solutions I see: 1) Handle the local port "by hand" instead of relying on the EFI implementation to chose it. 2) Save the StationPort as soon as it is known similarly to wha...

...cket > + * > + * @param:socket, the socket to open > + * > + * @out: error code, 0 on success, -1 on failure > + */ > +int core_udp_open(struct pxe_pvt_inode *socket) > +{ > + memset(&udata, 0, sizeof(udata)); > + udata.AcceptPromiscuous = TRUE; > + udata.AcceptAnyPort = TRUE; Why promiscuous? That seems to want to accept it on any local address and put the NIC into promiscuous mode, avoiding the default MAC filter in the NIC. This should be uncessary as we're not capturinig. > +/** > + * Establish a connection on an open socket > + * > + * @p...

On Thu, Nov 28, 2013 at 9:47 PM, Gene Cumm <gene.cumm at gmail.com> wrote: > On Thu, Nov 28, 2013 at 9:34 PM, Celelibi <celelibi at gmail.com> wrote: >> Without an assigned source port, Transmit function assign a random new >> source port to the packet being sent. It thus have to be set before >> calling Transmit if the source port have already been decided.

...ket to open > > + * > > + * @out: error code, 0 on success, -1 on failure > > + */ > > +int core_udp_open(struct pxe_pvt_inode *socket) > > +{ > > > + memset(&udata, 0, sizeof(udata)); > > + udata.AcceptPromiscuous = TRUE; > > + udata.AcceptAnyPort = TRUE; > > Why promiscuous? That seems to want to accept it on any local address > and put the NIC into promiscuous mode, avoiding the default MAC filter > in the NIC. This should be uncessary as we're not capturinig. If I remember correctly, this was required for TFTP boot bec...

Op 2013-11-08 om 10:48 schreef H. Peter Anvin: > Thank you for posting the pcap files, by the way. Analyzing them in > Wireshark is so much nicer than reading the text output of tcpdump. Fetch the new capture with wget -O pcap2.zip http://ge.tt/api/1/files/136167y/0/blob?downlad The capture[1] shows two[2] TFTP transmissions of sles113/syslinux.efi, but nothing after that. As if

...diff --git a/efi/udp.c b/efi/udp.c index 59bb426..60a8fe9 100644 --- a/efi/udp.c +++ b/efi/udp.c @@ -41,8 +41,7 @@ int core_udp_open(struct pxe_pvt_inode *socket) udp = (EFI_UDP4 *)udp_reader->this; memset(&udata, 0, sizeof(udata)); - udata.AcceptPromiscuous = TRUE; - udata.AcceptAnyPort = TRUE; + udata.AcceptBroadcast = TRUE; status = uefi_call_wrapper(udp->Configure, 2, udp, &udata); if (status != EFI_SUCCESS) @@ -50,6 +49,18 @@ int core_udp_open(struct pxe_pvt_inode *socket) socket->net.efi.binding = b; + /* + * Save the random local port...

I did change both instances of txdata->GatewayAddress, but I think something may be wrong in my toolchain. If I extract syslinux.zip and attempt to make from there (without modifying any files), the cached DHCP packet isn't read on the same subnet. I receive "Succeed to download NBP file." twice before anything from syslinux is loaded, so I believe the duplicate request for

...&Udp4Protocol); - if (!b) - return; + udp_io = (UDP_IO*)socket->net.efi.context; + + if(!udp_io->known_TID) + udp_io->known_TID=remote_port; + + udp_io->Config_Udp4.AcceptBroadcast = FALSE; + udp_io->Config_Udp4.AcceptPromiscuous = FALSE; + udp_io->Config_Udp4.AcceptAnyPort = FALSE; + udp_io->Config_Udp4.AllowDuplicatePort = FALSE; + udp_io->Config_Udp4.TypeOfService = 0; + udp_io->Config_Udp4.TimeToLive = 64; + udp_io->Config_Udp4.DoNotFragment = FALSE; + udp_io->Config_Udp4.ReceiveTimeout = 0; + udp_io->Config_Udp4.Transm...

The variable 'ok' is never used and generates a warning. Remove it. Also ntfssect.c is designed to be compiled in non Unicode mode when using MSVC compilers, so remove all ambiguity about it (LPCTSTR -> LPCSTR, use of 'A' API calls) so that it doesn't break when compiled in Unicode mode, which is what Rufus uses with MSVC. -------------- next part --------------