Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

To: Robert Elz <kre%munnari.OZ.AU@localhost>
Subject: Re: CVS commit: src/external/bsd/dhcpcd/dist/src
From: Martin Husemann <martin%duskware.de@localhost>
Date: Fri, 3 Aug 2018 15:54:24 +0200

On Fri, Aug 03, 2018 at 08:28:55PM +0700, Robert Elz wrote:
> Where is the signed arithmetic that was supposedly a probem?

Ah, stupid C integer promotion rules. uint16_t is promoted to int
here, not unsigned int or size_t. The cast makes all operands the same
type and no promotion happens.

Martin

Follow-Ups:
- Re: CVS commit: src/external/bsd/dhcpcd/dist/src
  - From: Valery Ushakov

References:
- Re: CVS commit: src/external/bsd/dhcpcd/dist/src
  - From: Martin Husemann
- Re: CVS commit: src/external/bsd/dhcpcd/dist/src
  - From: Robert Elz
- Re: CVS commit: src/external/bsd/dhcpcd/dist/src
  - From: Kamil Rytarowski
- Re: CVS commit: src/external/bsd/dhcpcd/dist/src
  - From: Robert Elz

Prev by Date: Re: CVS commit: src/external/bsd/dhcpcd/dist/src
Next by Date: Re: CVS commit: src/external/bsd/dhcpcd/dist/src
Previous by Thread: Re: CVS commit: src/external/bsd/dhcpcd/dist/src
Next by Thread: Re: CVS commit: src/external/bsd/dhcpcd/dist/src
Indexes:

Home | Main Index | Thread Index | Old Index