36

In K&R (The C Programming Language 2nd Edition) chapter 5 I read the following:

First, pointers may be compared under certain circumstances. If p and q point to members of the same array, then relations like ==, !=, <, >=, etc. work properly.

Which seems to imply that only pointers pointing to the same array can be compared.

However when I tried this code

    char t = 't';
    char *pt = &t;
    char x = 'x';
    char *px = &x;

    printf("%d\n", pt > px);

1 is printed to the screen.

First of all, I thought I would get undefined or some type or error, because pt and px aren't pointing to the same array (at least in my understanding).

Also is pt > px because both pointers are pointing to variables stored on the stack, and the stack grows down, so the memory address of t is greater than that of x? Which is why pt > px is true?

I get more confused when malloc is brought in. Also in K&R in chapter 8.7 the following is written:

There is still one assumption, however, that pointers to different blocks returned by sbrk can be meaningfully compared. This is not guaranteed by the standard which permits pointer comparisons only within an array. Thus this version of malloc is portable only among machines for which the general pointer comparison is meaningful.

I had no issue comparing pointers that pointed to space malloced on the heap to pointers that pointed to stack variables.

For example, the following code worked fine, with 1 being printed:

    char t = 't';
    char *pt = &t;
    char *px = malloc(10);
    strcpy(px, pt);
    printf("%d\n", pt > px);

Based on my experiments with my compiler, I'm being led to think that any pointer can be compared with any other pointer, regardless of where they individually point. Moreover, I think pointer arithmetic between two pointers is fine, no matter where they individually point because the arithmetic is just using the memory addresses the pointers store.

Still, I am confused by what I am reading in K&R.

The reason I'm asking is because my prof. actually made it an exam question. He gave the following code:

struct A {
    char *p0;
    char *p1;
};

int main(int argc, char **argv) {
    char a = 0;
    char *b = "W";
    char c[] = [ 'L', 'O', 'L', 0 ];

   struct A p[3];
    p[0].p0 = &a;
    p[1].p0 = b;
    p[2].p0 = c;

   for(int i = 0; i < 3; i++) {
        p[i].p1 = malloc(10);
        strcpy(p[i].p1, p[i].p0);
    }
}

What do these evaluate to:

  1. p[0].p0 < p[0].p1
  2. p[1].p0 < p[1].p1
  3. p[2].p0 < p[2].p1

The answer is 0, 1, and 0.

(My professor does include the disclaimer on the exam that the questions are for a Ubuntu Linux 16.04, 64-bit version programming environment)

(editor's note: if SO allowed more tags, that last part would warrant , , and maybe . If the point of the question / class was specifically low-level OS implementation details, rather than portable C.)

21
  • 17
    You are maybe confusing what is valid in C with what is safe in C. Comparing two pointers to the same type can always be done (checking for equality, for example) however, using pointer arithmetic and comparing > and < is only safe when used within a given array (or memory block). Dec 29, 2019 at 2:07
  • 14
    As an aside, you should not be learning C from K&R. For a start, the language has been through a lot of changes since then. And, to be honest, the example code in there was from a time when terseness rather than readabilty was valued.
    – paxdiablo
    Dec 29, 2019 at 12:15
  • 5
    No, it isn't guaranteed to work. It can fail in practice on machines with segmented memory models. See Does C have an equivalent of std::less from C++? On most modern machines, it will happen to work despite UB. Dec 29, 2019 at 16:05
  • 6
    @Adam: Close, but this is actually UB (unless the compiler the OP was using, GCC, does choose to define it. It might). But UB doesn't mean "definitely explodes"; one of the possible behaviours for UB is working the way you expected!! This is what makes UB so nasty; it can work right in a debug build and fail with optimization enabled, or vice versa, or break depending on surrounding code. Comparing other pointers will still give you an answer, but the language doesn't define what that answer will mean (if anything). No, crashing is allowed. It's truly UB. Dec 29, 2019 at 17:20
  • 3
    @Adam: Oh yes, nevermind the first part of my comment, I misread yours. But you claim Comparing other pointers will still give you an answer. That's not true. That would be an unspecified result, not full UB. UB is much worse and means your program could segfault or SIGILL if execution reaches that statement with those inputs (at any point before or after that actually happens). (Only plausible on x86-64 if the UB is visible at compile time, but in general anything can happen.) Part of the point of UB is to let the compiler make "unsafe" assumptions while generating asm. Dec 29, 2019 at 18:48

7 Answers 7

36

According to the C11 standard, the relational operators <, <=, >, and >= may only be used on pointers to elements of the same array or struct object. This is spelled out in section 6.5.8p5:

When two pointers are compared, the result depends on the relative locations in the address space of the objects pointed to. If two pointers to object types both point to the same object, or both point one past the last element of the same array object, they compare equal. If the objects pointed to are members of the same aggregate object,pointers to structure members declared later compare greater than pointers to members declared earlier in the structure, and pointers to array elements with larger subscript values compare greater than pointers to elements of the same array with lower subscript values. All pointers to members of the same union object compare equal. If the expression P points to an element of an array object and the expression Q points to the last element of the same array object, the pointer expression Q+1 compares greater than P. In all other cases, the behavior is undefined.

Note that any comparisons that do not satisfy this requirement invoke undefined behavior, meaning (among other things) that you can't depend on the results to be repeatable.

In your particular case, for both the comparison between the addresses of two local variables and between the address of a local and a dynamic address, the operation appeared to "work", however the result could change by making a seemingly unrelated change to your code or even compiling the same code with different optimization settings. With undefined behavior, just because the code could crash or generate an error doesn't mean it will.

As an example, an x86 processor running in 8086 real mode has a segmented memory model using a 16-bit segment and a 16-bit offset to build a 20-bit address. So in this case an address doesn't convert exactly to an integer.

The equality operators == and != however do not have this restriction. They can be used between any two pointers to compatible types or NULL pointers. So using == or != in both of your examples would produce valid C code.

However, even with == and != you could get some unexpected yet still well-defined results. See Can an equality comparison of unrelated pointers evaluate to true? for more details on this.

Regarding the exam question given by your professor, it makes a number of flawed assumptions:

  • A flat memory model exists where there is a 1-to-1 correspondence between an address and an integer value.
  • That the converted pointer values fit inside an integer type.
  • That the implementation simply treats pointers as integers when performing comparisons without exploiting the freedom given by undefined behavior.
  • That a stack is used and that local variables are stored there.
  • That a heap is used to pull allocated memory from.
  • That the stack (and therefore local variables) appears at a higher address than the heap (and therefore allocated objects).
  • That string constants appear at a lower address then the heap.

If you were to run this code on an architecture and/or with a compiler that does not satisfy these assumptions then you could get very different results.

Also, both examples also exhibit undefined behavior when they call strcpy, since the right operand (in some cases) points to a single character and not a null terminated string, resulting in the function reading past the bounds of the given variable.

16
  • 3
    @Shisui Even given that, you still shouldn't depend on the results. Compilers can get very aggressive when it comes to optimization and will use undefined behavior as an opportunity to do so. It's possible that using a different compiler and/or different optimization settings can generate different output.
    – dbush
    Dec 29, 2019 at 15:05
  • 2
    @Shisui: It will in general happen to work on machines with a flat memory model, such as x86-64. Some compilers for such systems might even define the behaviour in their documentation. But if not, then "insane" behaviour can happen because of compile-time-visible UB. (In practice I don't think anyone wants that so it's not something mainstream compilers look for and "try to break".) Dec 29, 2019 at 16:31
  • 1
    Like if a compiler sees that one path of execution would lead to < between malloc result and a local variable (automatic storage, i.e. stack), it could assume that path of execution is never taken and just compile the whole function to a ud2 instruction (raises an illegal-instruction exception which the kernel will handle by delivering a SIGILL to the process). GCC / clang do this in practice for other kinds of UB, like falling off the end of a non-void function. godbolt.org is down right now it seems, but try copy/pasting int foo(){int x=2;} and note the lack of a ret Dec 29, 2019 at 16:31
  • 4
    @Shisui: TL:DR: it's not portable C, despite the fact that it happens to work fine on x86-64 Linux. Making assumptions about the results of the comparison is just crazy, though. If you aren't in the main thread, your thread stack will have been dynamically allocated using the same mechanism malloc uses to get more memory from the OS, so there's no reason to assume that your local vars (thread stack) are above malloc dynamically-allocated storage. Dec 29, 2019 at 16:38
  • 2
    @PeterCordes: What's needed is to recognize various aspects of behavior as "optionally defined", such that implementations may define them or not, at their leisure, but must indicate in testable fashion (e.g. predefined macro) if they do not do so. Additionally, instead of characterizing that any situation where the effects of an optimization would be observable as "Undefined Behavior", it would be far more useful to say that optimizers may regard certain aspects of behavior as "non-observable" if they indicate that they do so. For example, given int x,y;, an implementation...
    – supercat
    Dec 29, 2019 at 18:01
12

The primary issue with comparing pointers to two distinct arrays of the same type is that the arrays themselves need not be placed in a particular relative positioning--one could end up before and after the other.

First of all, I thought I would get undefined or some type or error, because pt an px aren't pointing to the same array (at least in my understanding).

No, the result is dependent on implementation and other unpredictable factors.

Also is pt>px because both pointers are pointing to variables stored on the stack, and the stack grows down, so the memory address of t is greater than that of x? Which is why pt>px is true?

There isn't necessarily a stack. When it exists, it need not to grow down. It could grow up. It could be non-contiguous in some bizarre way.

Moreover, I think pointer arithmetic between two pointers is fine, no matter where they individually point because the arithmetic is just using the memory addresses the pointers store.

Let's look at the C specification, §6.5.8 on page 85 which discusses relational operators (i.e. the comparison operators you're using). Note that this does not apply to direct != or == comparison.

When two pointers are compared, the result depends on the relative locations in the address space of the objects pointed to. ... If the objects pointed to are members of the same aggregate object, ... pointers to array elements with larger subscript values compare greater than pointers to elements of the same array with lower subscript values.

In all other cases, the behavior is undefined.

The last sentence is important. While I cut down some unrelated cases to save space, there's one case that's important to us: two arrays, not part of the same struct/aggregate object1, and we're comparing pointers to those two arrays. This is undefined behavior.

While your compiler just inserted some sort of CMP (compare) machine instruction which numerically compares the pointers, and you got lucky here, UB is a pretty dangerous beast. Literally anything can happen--your compiler could optimize out the whole function including visible side effects. It could spawn nasal demons.

1Pointers into two different arrays that are part of the same struct can be compared, since this falls under the clause where the two arrays are part of the same aggregate object (the struct).

2
  • 1
    More importantly, with t and x being defined in the same function, there's zero reason to assume anything about how a compiler targeting x86-64 will lay out locals in the stack frame for this function. The stack growing downward has nothing to do with declaration order of variables in one function. Even in separate functions, if one could inline into the other then the "child" function's locals could still mix with the parents. Dec 29, 2019 at 16:41
  • 1
    your compiler could optimize out the whole function including visible side effects Not an overstatement: for other kinds of UB (like falling off the end of a non-void function) g++ and clang++ really do that in practice: godbolt.org/z/g5vesB they assume that path of execution is not taken because it leads to UB, and compile any such basic blocks to an illegal instruction. Or to no instructions at all, just silently falling through to whatever asm is next if that function ever were called. (For some reason gcc doesn't do this, only g++). Dec 29, 2019 at 16:47
6

Then asked what

p[0].p0 < p[0].p1
p[1].p0 < p[1].p1
p[2].p0 < p[2].p1

Evaluate to. The answer is 0, 1, and 0.

These questions reduce to:

  1. Is the heap above or below the stack.
  2. Is the heap above or below the string literal section of the program.
  3. same as [1].

And the answer to all three is "implementation defined". Your prof's questions are bogus; they have based it in traditional unix layout:

<empty>
text
rodata
rwdata
bss
< empty, used for heap >
...
stack
kernel

but several modern unices (and alternative systems) do not conform to those traditions. Unless they prefaced the question with " as of 1992 "; make sure to give a -1 on the eval.

10
  • 3
    Not implementation defined, undefined! Think of it this way, the former may vary between implementations but the implementations should document how behaviour is decided. The latter means behaviour may vary in any manner and the implementation doesn't have to tell you squat :-)
    – paxdiablo
    Dec 29, 2019 at 14:31
  • 1
    @paxdiablo: According to Rationale by the authors of the Standard, "Undefined behavior... also identifies areas of possible conforming language extension: the implementor may augment the language by providing a definition of the officially undefined behavior." The Rationale further says "The goal is to give the programmer a fighting chance to make powerful C programs that are also highly portable, without seeming to demean perfectly useful C programs that happen not to be portable, thus the adverb strictly." Commercial compiler writers understand this, but some other compiler writers don't.
    – supercat
    Dec 29, 2019 at 16:40
  • There is another implementation defined aspect; pointer comparison is signed, so depending upon the machine / os / compiler, some addresses may be interpreted as negative. For example, a 32bit machine that placed the stack at 0xc<<28, would likely show the automatic variables at a lessor address than the heap or rodata.
    – mevets
    Dec 29, 2019 at 19:25
  • 1
    @mevets: Does the Standard specify any situation in which the signedness of pointers in comparisons would be observable? I would expect that if a 16-bit platform allows objects greater than 32768 bytes, and arr[] is such an object,, the Standard would mandate that arr+32768 compare greater than arr even if a signed pointer comparison would report otherwise.
    – supercat
    Dec 29, 2019 at 20:22
  • I don't know; the C standard is orbiting in Dante's ninth circle, praying for euthanasia. The OP specifically referenced K&R and an exam question. #UB is debris from a lazy working group.
    – mevets
    Dec 29, 2019 at 20:32
1

On almost any remotely-modern platform, pointers and integers have an isomorphic ordering relation, and pointers to disjoint objects are not interleaved. Most compilers expose this ordering to programmers when optimizations are disabled, but the Standard makes no distinction between platforms that have such an ordering and those that don't and does not require that any implementations expose such an ordering to the programmer even on platforms that would define it. Consequently, some compiler writers perform various kinds of optimizations and "optimizations" based upon an assumption that code will never compare use relational operators on pointers to different objects.

According to the published Rationale, the authors of the Standard intended that implementations extend the language by specifying how they will behave in situations the Standard characterizes as "Undefined Behavior" (i.e. where the Standard imposes no requirements) when doing so would be useful and practical, but some compiler writers would rather assume programs will never try to benefit from anything beyond what the Standard mandates, than allow programs to usefully exploit behaviors the platforms could support at no extra cost.

I'm not aware of any commercially-designed compilers that do anything weird with pointer comparisons, but as compilers move to the non-commercial LLVM for their back end, they're increasingly likely to process nonsensically code whose behavior had been specified by earlier compilers for their platforms. Such behavior isn't limited to relational operators, but can even affect equality/inequality. For example, even though the Standard specifies that a comparison between a pointer to one object and a "just past" pointer to an immediately-preceding object will compare equal, gcc and LLVM-based compilers are prone to generate nonsensical code if programs perform such comparisons.

As an example of a situation where even equality comparison behaves nonsensically in gcc and clang, consider:

extern int x[],y[];
int test(int i)
{
    int *p = y+i;
    y[0] = 4;
    if (p == x+10)
        *p = 1;
    return y[0];
}

Both clang and gcc will generate code that will always return 4 even if x is ten elements, y immediately follows it, and i is zero resulting in the comparison being true and p[0] being written with the value 1. I think what happens is that one pass of optimization rewrites the function as though *p = 1; were replaced with x[10] = 1;. The latter code would be equivalent if the compiler interpreted *(x+10) as equivalent to *(y+i), but unfortunately a downstream optimization stage recognizes that an access to x[10] would only defined if x had at least 11 elements, which would make it impossible for that access to affect y.

If compilers can get that "creative" with pointer equality scenario which is described by the Standard, I would not trust them to refrain from getting even more creative in cases where the Standard doesn't impose requirements.

0
0

It's simple: Comparing pointers does not make sense as memory locations for objects are never guaranteed to be in the same order as you declared them. The exception is arrays. &array[0] is lower than &array[1]. Thats what K&R points out. In practice struct member addresses are also in the order you declare them in my experience. No guarantees on that.... Another exception is if you compare a pointer for equal. When one pointer is equal to another you know it's pointing to the same object. Whatever it is. Bad exam question if you ask me. Depending on Ubuntu Linux 16.04, 64-bit version programming environment for an exam question ? Really ?

2
  • Technically, arrays aren't really an exception since you don't declare arr[0], arr[1], etc separately. You declare arr as a whole so the ordering of individual array elements is a different issue to that described in this question.
    – paxdiablo
    Dec 31, 2019 at 4:50
  • 1
    Structure elements are guaranteed to be in order, which guarantees that one may use memcpy to copy a contiguous part of a structure and affect all of the elements therein and not affect anything else. The Standard is sloppy about terminology as to what kinds of pointer arithmetic can be done with structures or malloc() allocated storage. The offsetof macro would be rather useless if one couldn't to the same kind of pointer arithmetic with the bytes of a struct as with a char[], but the Standard doesn't expressly say that the bytes of a struct are (or can be used as) an array object.
    – supercat
    Dec 31, 2019 at 18:05
-5

Pointers are just integers, like everything else in a computer. You absolutely can compare them with < and > and produce results without causing a program to crash. That said, the standard does not guarantee that those results have any meaning outside of array comparisons.

In your example of stack allocated variables, the compiler is free to allocate those variables to registers or stack memory addresses, and in any order it so choose. Comparisons such as < and > therefore won't be consistent across compilers or architectures. However, == and != aren't so restricted, comparing pointer equality is a valid and useful operation.

15
  • 2
    The word stack appears exactly zero times in the C11 standard. And undefined behaviour means anything can happen (including program crash).
    – paxdiablo
    Dec 29, 2019 at 12:09
  • 1
    @paxdiablo Did I say it did?
    – nickelpro
    Dec 29, 2019 at 12:20
  • 2
    You mentioned stack-allocated variables. There is no stack in the standard, that's just an implementation detail. The more serious issue with this answer is the contention you can compare pointers with no chance of a crash - that's just wrong.
    – paxdiablo
    Dec 29, 2019 at 14:13
  • 1
    @nickelpro: If one wishes to write code that is compatible with the optimizers in gcc and clang, it is necessary to jump through a lot of silly hoops. Both optimizers will aggressively seek out opportunities to draw inferences about what things will be accessed by pointers whenever there's any way the Standard can be twisted to justify them (and even sometimes when there isn't). Given int x[10],y[10],*p;, if code evaluates y[0], then evaluates p>(x+5) and writes *p without modifying p in the interim, and finally evaluates y[0] again, ...
    – supercat
    Dec 29, 2019 at 19:53
  • 1
    nickelpro, agree to agree to disagree but your answer is still fundamentally wrong. I liken your approach to that of the people that use (ch >= 'A' && ch <= 'Z') || (ch >= 'a' && ch <= 'z') instead of isalpha() because what sane implementation would have those characters discontinuous? The bottom line is that, even if no implementation you know has a problem, you should be coding to the standard as much as possible if you value portability. I do appreciate the label "standards maven" though, thanks for that. I may put in on my CV :-)
    – paxdiablo
    Dec 30, 2019 at 0:46
-5

What A Provocative Question!

Even cursory scanning of the responses and comments in this thread will reveal how emotive your seemingly simple and straight forward query turns out to be.

It should not be surprising.

Inarguably, misunderstandings around the concept and use of pointers represents a predominant cause of serious failures in programming in general.

Recognition of this reality is readily evident in the ubiquity of languages designed specifically to address, and preferably to avoid the challenges pointers introduce altogether. Think C++ and other derivatives of C, Java and its relations, Python and other scripts -- merely as the more prominent and prevalent ones, and more or less ordered in severity of dealing with the issue.

Developing a deeper understanding of the principles underlying, therefore must be pertinent to every individual that aspires to excellence in programming -- especially at the systems level.

I imagine this is precisely what your teacher means to demonstrate.

And the nature of C makes it a convenient vehicle for this exploration. Less clearly than assembly -- though perhaps more readily comprehensible -- and still far more explicitly than languages based on deeper abstraction of the execution environment.

Designed to facilitate deterministic translation of the programmer’s intent into instructions that machines can comprehend, C is a system level language. While classified as high-level, it really belongs in a ‘medium’ category; but since none such exists, the ‘system’ designation has to suffice.

This characteristic is largely responsible for making it a language of choice for device drivers, operating system code, and embedded implementations. Furthermore, a deservedly favoured alternative in applications where optimal efficiency is paramount; where that means the difference between survival and extinction, and therefore is a necessity as opposed to a luxury. In such instances, the attractive convenience of portability loses all its allure, and opting for the lack-lustre performance of the least common denominator becomes an unthinkably detrimental option.

What makes C -- and some of its derivatives -- quite special, is that it allows its users complete control -- when that is what they desire -- without imposing the related responsibilities upon them when they do not. Nevertheless, it never offers more than the thinnest of insulations from the machine, wherefore proper use demands exacting comprehension of the concept of pointers.

In essence, the answer to your question is sublimely simple and satisfyingly sweet -- in confirmation of your suspicions. Provided, however, that one attaches the requisite significance to every concept in this statement:

  • The acts of examining, comparing and manipulating pointers are always and necessarily valid, while the conclusions derived from the result depends on the validity of the values contained, and thus need not be.

The former is both invariably safe and potentially proper, while the latter can only ever be proper when it has been established as safe. Surprisingly -- to some -- so establishing the validity of the latter depends on and demands the former.

Of course, part of the confusion arises from the effect of the recursion inherently present within the principle of a pointer -- and the challenges posed in differentiating content from address.

You have quite correctly surmised,

I'm being led to think that any pointer can be compared with any other pointer, regardless of where they individually point. Moreover, I think pointer arithmetic between two pointers is fine, no matter where they individually point because the arithmetic is just using the memory addresses the pointers store.

And several contributors have affirmed: pointers are just numbers. Sometimes something closer to complex numbers, but still no more than numbers.

The amusing acrimony in which this contention has been received here reveals more about human nature than programming, but remains worthy of note and elaboration. Perhaps we will do so later...

As one comment begins to hint; all this confusion and consternation derives from the need to discern what is valid from what is safe, but that is an oversimplification. We must also distinguish what is functional and what is reliable, what is practical and what may be proper, and further still: what is proper in a particular circumstance from what may be proper in a more general sense. Not to mention; the difference between conformity and propriety.

Toward that end, we first need to appreciate precisely what a pointer is.

  • You have demonstrated a firm grip on the concept, and like some others may find these illustrations patronizingly simplistic, but the level of confusion evident here demands such simplicity in clarification.

As several have pointed out: the term pointer is merely a special name for what is simply an index, and thus nothing more than any other number.

This should already be self-evident in consideration of the fact that all contemporary mainstream computers are binary machines that necessarily work exclusively with and on numbers. Quantum computing may change that, but that is highly unlikely, and it has not come of age.

Technically, as you have noted, pointers are more accurately addresses; an obvious insight that naturally introduces the rewarding analogy of correlating them with the ‘addresses’ of houses, or plots on a street.

  • In a flat memory model: the entire system memory is organized in a single, linear sequence: all houses in the city lie on the same road, and every house is uniquely identified by its number alone. Delightfully simple.

  • In segmented schemes: a hierarchical organization of numbered roads is introduced above that of numbered houses so that composite addresses are required.

    • Some implementations are still more convoluted, and the totality of distinct ‘roads’ need not sum to a contiguous sequence, but none of that changes anything about the underlying.
    • We are necessarily able to decompose every such hierarchical link back into a flat organization. The more complex the organization, the more hoops we will have to hop through in order to do so, but it must be possible. Indeed, this also applies to ‘real mode’ on x86.
    • Otherwise the mapping of links to locations would not be bijective, as reliable execution -- at the system level -- demands that it MUST be.
      • multiple addresses must not map to singular memory locations, and
      • singular addresses must never map to multiple memory locations.

Bringing us to the further twist that turns the conundrum into such a fascinatingly complicated tangle. Above, it was expedient to suggest that pointers are addresses, for the sake of simplicity and clarity. Of course, this is not correct. A pointer is not an address; a pointer is a reference to an address, it contains an address. Like the envelope sports a reference to the house. Contemplating this may lead you to glimpse what was meant with the suggestion of recursion contained in the concept. Still; we have only so many words, and talking about the addresses of references to addresses and such, soon stalls most brains at an invalid op-code exception. And for the most part, intent is readily garnered from context, so let us return to the street.

Postal workers in this imaginary city of ours are much like the ones we find in the ‘real’ world. No one is likely to suffer a stroke when you talk or enquire about an invalid address, but every last one will balk when you ask them to act on that information.

Suppose there are only 20 houses on our singular street. Further pretend that some misguided, or dyslexic soul has directed a letter, a very important one, to number 71. Now, we can ask our carrier Frank, whether there is such an address, and he will simply and calmly report: no. We can even expect him to estimate how far outside the street this location would lie if it did exist: roughly 2.5 times further than the end. None of this will cause him any exasperation. However, if we were to ask him to deliver this letter, or to pick up an item from that place, he is likely to be quite frank about his displeasure, and refusal to comply.

Pointers are just addresses, and addresses are just numbers.

Verify the output of the following:

void foo( void *p ) {
   printf(“%p\t%zu\t%d\n”, p, (size_t)p, p == (size_t)p);
}

Call it on as many pointers as you like, valid or not. Please do post your findings if it fails on your platform, or your (contemporary) compiler complains.

Now, because pointers are simply numbers, it is inevitably valid to compare them. In one sense this is precisely what your teacher is demonstrating. All of the following statements are perfectly valid -- and proper! -- C, and when compiled will run without encountering problems, even though neither pointer need be initialized and the values they contain therefore may be undefined:

  • We are only calculating result explicitly for the sake of clarity, and printing it to force the compiler to compute what would otherwise be redundant, dead code.
void foo( size_t *a, size_t *b ) {
   size_t result;
   result = (size_t)a;
   printf(“%zu\n”, result);
   result = a == b;
   printf(“%zu\n”, result);
   result = a < b;
   printf(“%zu\n”, result);
   result = a - b;
   printf(“%zu\n”, result);
}

Of course, the program is ill-formed when either a or b is undefined (read: not properly initialized) at the point of testing, but that is utterly irrelevant to this part of our discussion. These snippets, as too the following statements, are guaranteed -- by the ‘standard’ -- to compile and run flawlessly, notwithstanding the IN-validity of any pointer involved.

Problems only arise when an invalid pointer is dereferenced. When we ask Frank to pick up or deliver at the invalid, non-existent address.

Given any arbitrary pointer:

int *p;

While this statement must compile and run:

printf(“%p”, p);

... as must this:

size_t foo( int *p ) { return (size_t)p; }

... the following two, in stark contrast, will still readily compile, but fail in execution unless the pointer is valid -- by which we here merely mean that it references an address to which the present application has been granted access:

printf(“%p”, *p);
size_t foo( int *p ) { return *p; }

How subtle the change? The distinction lies in the difference between the value of the pointer -- which is the address, and the value of the contents: of the house at that number. No problem arises until the pointer is dereferenced; until an attempt is made to access the address it links to. In trying to deliver or pick up the package beyond the stretch of the road...

By extension, the same principle necessarily applies to more complex examples, including the aforementioned need to establish the requisite validity:

int* validate( int *p, int *head, int *tail ) { 
    return p >= head && p <= tail ? p : NULL; 
}

Relational comparison and arithmetic offer identical utility to testing equivalence, and are equivalently valid -- in principle. However, what the results of such computation would signify, is a different matter entirely -- and precisely the issue addressed by the quotations you included.

In C, an array is a contiguous buffer, an uninterrupted linear series of memory locations. Comparison and arithmetic applied to pointers that reference locations within such a singular series are naturally, and obviously meaningful in relation both to each other, and to this ‘array’ (which is simply identified by the base). Precisely the same applies to every block allocated through malloc, or sbrk. Because these relationships are implicit, the compiler is able to establish valid relationships between them, and therefore can be confident that calculations will provide the answers anticipated.

Performing similar gymnastics on pointers that reference distinct blocks or arrays do not offer any such inherent, and apparent utility. The more so since whatever relation exists at one moment may be invalidated by a reallocation that follows, wherein that is highly likely to change, even be inverted. In such instances the compiler is unable to obtain the necessary information to establish the confidence it had in the previous situation.

You, however, as the programmer, may have such knowledge! And in some instances are obliged to exploit that.

There ARE, therefore, circumstances in which EVEN THIS is entirely VALID and perfectly PROPER.

In fact, that is exactly what malloc itself has to do internally when time comes to try merging reclaimed blocks -- on the vast majority of architectures. The same is true for the operating system allocator, like that behind sbrk; if more obviously, frequently, on more disparate entities, more critically -- and relevant also on platforms where this malloc may not be. And how many of those are not written in C?

The validity, security and success of an action is inevitably the consequence of the level of insight upon which it is premised and applied.

In the quotes you have offered, Kernighan and Ritchie are addressing a closely related, but nonetheless separate issue. They are defining the limitations of the language, and explaining how you may exploit the capabilities of the compiler to protect you by at least detecting potentially erroneous constructs. They are describing the lengths the mechanism is able -- is designed -- to go to in order to assist you in your programming task. The compiler is your servant, you are the master. A wise master, however, is one that is intimately familiar with the capabilities of his various servants.

Within this context, undefined behaviour serves to indicate potential danger and the possibility of harm; not to imply imminent, irreversible doom, or the end of the world as we know it. It simply means that we -- ‘meaning the compiler’ -- are not able to make any conjecture about what this thing may be, or represent and for this reason we choose to wash our hands of the matter. We will not be held accountable for any misadventure that may result from the use, or mis-use of this facility.

In effect, it simply says: ‘Beyond this point, cowboy: you are on your own...’

Your professor is seeking to demonstrate the finer nuances to you.

Notice what great care they have taken in crafting their example; and how brittle it still is. By taking the address of a, in

p[0].p0 = &a;

the compiler is coerced into allocating actual storage for the variable, rather than placing it in a register. It being an automatic variable, however, the programmer has no control over where that is assigned, and so unable to make any valid conjecture about what would follow it. Which is why a must be set equal to zero for the code to work as expected.

Merely changing this line:

char a = 0;

to this:

char a = 1;  // or ANY other value than 0

causes the behaviour of the program to become undefined. At minimum, the first answer will now be 1; but the problem is far more sinister.

Now the code is inviting of disaster.

While still perfectly valid and even conforming to the standard, it now is ill-formed and although sure to compile, may fail in execution on various grounds. For now there are multiple problems -- none of which the compiler is able to recognize.

strcpy will start at the address of a, and proceed beyond this to consume -- and transfer -- byte after byte, until it encounters a null.

The p1 pointer has been initialized to a block of exactly 10 bytes.

  • If a happens to be placed at the end of a block and the process has no access to what follows, the very next read -- of p0[1] -- will elicit a segfault. This scenario is unlikely on the x86 architecture, but possible.

  • If the area beyond the address of a is accessible, no read error will occur, but the program still is not saved from misfortune.

  • If a zero byte happens to occur within the ten starting at the address of a, it may still survive, for then strcpy will stop and at least we will not suffer a write violation.

  • If it is not faulted for reading amiss, but no zero byte occurs in this span of 10, strcpy will continue and attempt to write beyond the block allocated by malloc.

    • If this area is not owned by the process, the segfault should immediately be triggered.

    • The still more disastrous -- and subtle --- situation arises when the following block is owned by the process, for then the error cannot be detected, no signal can be raised, and so it may ‘appear’ still to ‘work’, while it actually will be overwriting other data, your allocator’s management structures, or even code (in certain operating environments).

This is why pointer related bugs can be so hard to track. Imagine these lines buried deep within thousands of lines of intricately related code, that someone else has written, and you are directed to delve through.

Nevertheless, the program must still compile, for it remains perfectly valid and standard conformant C.

These kinds of errors, no standard and no compiler can protect the unwary against. I imagine that is exactly what they are intending to teach you.

Paranoid people constantly seek to change the nature of C to dispose of these problematic possibilities and so save us from ourselves; but that is disingenuous. This is the responsibility we are obliged to accept when we choose to pursue the power and obtain the liberty that more direct and comprehensive control of the machine offers us. Promoters and pursuers of perfection in performance will never accept anything less.

Portability and the generality it represents is a fundamentally separate consideration and all that the standard seeks to address:

This document specifies the form and establishes the interpretation of programs expressed in the programming language C. Its purpose is to promote portability, reliability, maintainability, and efficient execution of C language programs on a variety of computing systems.

Which is why it is perfectly proper to keep it distinct from the definition and technical specification of the language itself. Contrary to what many seem to believe generality is antithetical to exceptional and exemplary.

To conclude:

  • Examining and manipulating pointers themselves is invariably valid and often fruitful. Interpretation of the results, may, or may not be meaningful, but calamity is never invited until the pointer is dereferenced; until an attempt is made to access the address linked to.

Were this not true, programming as we know it -- and love it -- would not have been possible.

34
  • 3
    This answer is unfortunately inherently invalid. You can't reason anything about undefined behaviour. The comparison need not be done at machine level. Dec 29, 2019 at 9:58
  • 6
    Ghii, actually no. If you look at C11 Annex J and 6.5.8, the act of comparison itself is UB. Dereferencing is a separate issue.
    – paxdiablo
    Dec 29, 2019 at 14:22
  • 6
    No, UB can still be harmful even before a pointer is dereferenced. A compiler is free to completely optimize a function with UB into a single NOP, even though this obviously changes visible behavior.
    – nanofarad
    Dec 29, 2019 at 14:23
  • 2
    @Ghii, Annex J (the bit I mentioned) is the list of things that are undefined behaviour, so I'm not sure how that supports your argument :-) 6.5.8 explicitly calls out the comparison as UB. For your comment to supercat, there's no comparison going on when you print a pointer so you're probably right that it will not crash. But that's not what the OP was asking about. 3.4.3 is also a section you should look at: it defines UB as behaviour "for which this International Standard imposes no requirements".
    – paxdiablo
    Dec 30, 2019 at 0:53
  • 3
    @GhiiVelte, you keep stating things that are just plainly wrong, despite that being pointed out to you. Yes, the snippet you posted must compile but your contention that it run without hitch is incorrect. I suggest you actually read the standard, particularly (in this case) C11 6.5.6/9, keeping in mind that the word "shall" indicates a requirementL "When two pointers are subtracted, both shall point to elements of the same array object, or one past the last element of the array object".
    – paxdiablo
    Dec 30, 2019 at 5:10

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.