I am getting error in this code as "invalid indirection"

Question

I am trying to dynamically allocate a contiguous block of memory, store some integer value and display it.

#include<stdio.h>
#include<conio.h>
void main()
{     int i;
      int *ptr;
      ptr=(void *)malloc(sizeof(int)*5); //allocation of memory

      for(i=0;i<5;i++)
              {     scanf("%d",&ptr[i]);
              }    
      for(i=0;i<5;i++)
              {    printf("%d",*ptr[i]); //error is found here``                     
              }
}    }

dont cast malloc standard warning and check if memory allocation was successful or not — Sourav Kanta, Jun 20 '15 at 04:27
`int main(void)` please and unless `conio.h` pulls in a header that declares `malloc()` you should `#include `. — lord.garbage, Jun 20 '15 at 04:59

Sourav Kanta · Accepted Answer · 2015-06-20T05:17:40.620

1

ptr[i] means the value at address (ptr+i) so *ptr[i] is meaningless.You should remove the *

Your corrected code should be :

 #include<stdio.h>
 #include<stdlib.h>
 #include<conio.h>
 int main()
 { 
  int i;
  int *ptr;
  ptr=malloc(sizeof(int)*5); //allocation of memory

  for(i=0;i<5;i++)
          {     scanf("%d",&ptr[i]);    
           }    
  for(i=0;i<5;i++)
          {    printf("%d",ptr[i]); //loose the *
          }
  return 0;
}     //loose extra }

edited Jun 20 '15 at 05:17

answered Jun 20 '15 at 04:42

Sourav Kanta

2,727
1
18
29

`void main` should be `int main` – M.M Jun 20 '15 at 05:07
@MattMcNabb I dont understand why that is a compulsion.Would you please explain > – Sourav Kanta Jun 20 '15 at 05:10
**Thank you** @MattMcNabb – Sourav Kanta Jun 20 '15 at 05:17

I am getting error in this code as "invalid indirection"

1 Answers1