You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 23 Next »

While it has been common practice to use integers and pointers interchangeably in C, pointer to integer and integer to pointer conversions are implementation-defined.

According to C99 [[ISO/IEC 9899-1999]], the only value that can be considered interchangeable between pointers and integers is the constant 0. Except in this case, conversions between integers and pointers may have undesired consequences depending on the implementation:

An integer may be converted to any pointer type. Except as previously specified, the result is implementation-defined, might not be correctly aligned, might not point to an entity of the referenced type, and might be a trap representation.

Any pointer type may be converted to an integer type.  Except as previously specified, the result is implementation-defined.  If the result cannot be represented in the integer type, the behavior is undefined.  The result need not be in the range of values of any integer type.

These issues arise because the mapping functions for converting a pointer to an integer or an integer to a pointer must be consistent with the addressing structure of the execution environment.  For example, not all machines have a flat memory model.

It is sometimes necessary in low level kernel or graphics code to access memory at a specific location, requiring a literal integer to pointer conversion such as the following.  It should be recognized that this is specific to a particular type of machine, and should therefore be done only when necessary.

unsigned int *ptr = 0xcfcfcfcf;

Non-Compliant Code Example

In this non-compliant code example, the pointer ptr is converted to an integer value.  Both a pointer and an int are assumed to be 32 bits.  The upper 9 bits of the number are used to hold a flag value and the result is converted back into a pointer.

char *ptr;
/* ... */
unsigned int number = ptr;
number = (number & 0x7fffff) | (flag << 23);
ptr = number;

A scheme similar to this was actually used in early versions of Emacs, sacrificing its portability and its ability to edit files larger than 8MB.

Compliant Solution

Saving a few bits of storage is not so important that it is worth writing nonportable code.  A struct can be used to provide room for both the pointer and the flag value.  This is portable to machines of different word sizes, both smaller and larger than 32 bits, and works even when pointers cannot be represented in any integer type.

struct ptrflag {
  char *pointer;
  unsigned int flag :9;
} ptrflag;
/* ... */
ptrflag.pointer = ptr;
ptrflag.flag = flag;

Non-Compliant Code Example

The following code tries to determine whether two character pointers are aligned to each other.

char *ptr1;
char *ptr2;
/* ... */
if (((unsigned)ptr1 & 3) == ((unsigned)ptr2 & 3)) {
  /* ... */

Although this is likely to work on many architectures, it does not provide maximum portability.

Compliant Solution

On machines where pointers can be represented as integers, the types intptr_t and uintptr_t are provided for that purpose.  They are only guaranteed to be able to receive void pointers.

#include <stdint.h>
/* ... */
char *ptr1;
char *ptr2;
/* ... */
if (((uintptr_t)(void *)ptr1 & ALIGN_BITS) == ((uintptr_t)(void *)ptr2 & ALIGN_BITS)) {
  /* ... */

The header inttypes.h can be used instead of stdint.h to get the integer types in a hosted environment.

Risk Analysis

Converting from pointer to integer or vice versa results in unportable code and may create unexpected pointers to invalid memory locations.

Recommendation

Severity

Likelihood

Remediation Cost

Priority

Level

INT15-A

1 (low)

2 (probable)

1 (high)

P2

L3

References

[[ISO/IEC 9899-1999]] Section 6.3.2.3, "Pointers"

  • No labels