169

Today I reached page 167 of The C Programming Language (second edition Brian W. Kernighan & Dennis M. Ritchie) and found that the author says I must cast malloc. Here is the part from the book:

7.8.5 Storage Management

The functions malloc and calloc obtain blocks of memory dynamically.

void *malloc(size_t n)

returns a pointer to n bytes of uninitialized storage, or NULL if the request cannot be satisfied.

void *calloc(size_t n, size_t size)

returns a pointer to enough free space for an array of n objects of the specified size, or NULL if the request cannot be satisfied. The storage is initialized to zero. The pointer returned by malloc or calloc has the proper alignment for the object in question, but it must be cast into the appropriate type, as in

int *ip;
ip = (int *) calloc(n, sizeof(int));

I already know that malloc (and its family) returns type void*, and there are good explanations why not to cast malloc.

But my question is: Why does the book say I should cast it?

8
  • 133
    Because the book is old. Sep 18, 2015 at 12:38
  • 12
    Because even the Sun has its dark spots, would be my answer. In other words, the book is wrong. It could be that the text predates the semantics for void *, and wasn't updated. See also this answer.
    – unwind
    Sep 18, 2015 at 12:38
  • 8
    @Michi The book has many factual and typographic errors (google K&R errata), it is only somewhat compatible with the C90 standard, it does not address the current C standard nor any changes in the language since 1990. Worst of all it is filled with bad programming practice, bad style and code which relies on poorly-specified behavior. All of which you have to unlearn if you become a professional C programmer.
    – Lundin
    Sep 18, 2015 at 13:04
  • 9
    ...and contrast this with Why does the compiler complain when I do not cast the result of malloc? So, for C - don't cast. For C++ - cast, but don't use malloc because it's NOT C++ - except when you have to - but you shouldn't - except...AGGGHHHHHH!!!!! :-) Sep 18, 2015 at 17:29
  • 2
    @Mandrill have you read my Question ? I had to edit my Question for you.
    – Michi
    Sep 18, 2015 at 18:15

1 Answer 1

229

From http://computer-programming-forum.com/47-c-language/a9c4a586c7dcd3fe.htm:

In pre-ANSI C -- as described in K&R-1 -- malloc() returned a char * and it was necessary to cast its return value in all cases where the receiving variable was not also a char *. The new void * type in Standard C makes these contortions unnecessary.

To save anybody from the embarrassment of leaping needlessly to the defence of K&R-2, I asked Dennis Ritchie for an opinion that I could quote on the validity of the sentence cited above from page 142. He replied:

In any case, now that I reread the stuff on p. 142, I think it's wrong; it's written in such a way that it's not just defensive against earlier rules, it misrepresents the ANSI rules.

11
  • 24
    So the return type of malloc was char and not void. Thank you.
    – Michi
    Sep 18, 2015 at 12:57
  • 11
    It's also mentioned in the errata web.archive.org/web/20150205025553/http://cm.bell-labs.com/cm/…
    – nos
    Sep 18, 2015 at 13:17
  • 18
    You mustn't read this book without that errata printed on a paper(s) next to you.
    – Lundin
    Sep 18, 2015 at 13:19
  • 5
    @Michi - no, the return type was char*, not char. The two are vastly different. Sep 18, 2015 at 13:41
  • 20
    @alk: exactly. By reading K&R you're listening to two gurus, which is ample ;-) Sep 18, 2015 at 15:36

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