How to create dynamic array of structs using realloc in function

The central problem can perhaps be best summarized by these two excerpts from your code:

    temp = realloc(*tab, (*PicCounter) *sizeof(*temp));

[...]

        **tab = temp;

You are reallocating the space pointed to by the value of *tab, but on success, you're storing the pointer to the new space in **tab, which is a different location. Regardless of which of those is the right location for the pointer, the combination is certainly wrong.

In fact, however, given the way you are calling your function ...

struct Picture *tabofpics;

[...]

AddPicture(&tabofpics,&piccounter);

... the type of the actual argument is struct Picture **. And that's appropriate for your purpose, so you should adjust the function definition to match:

int AddPicture(struct Picture **tab, int *PicCounter)

. Moreover, you should ensure that a matching function declaration is present in a header file that is #included into the source file containing the definition of AddPicture and into every source file containing a call to that function. That, plus ensuring that compiler warnings are turned up, should empower the compiler to help you diagnose mismatches between the definition and the uses of this function.

Having done that, the expression *tab in function AddPicture() will refer to the same object that the expression tabofpics (no *) refers to in main(). That appears to be consistent with how you're actually using it, with the exception of the mismatch I described at the beginning.

There are other reasonable criticisms of your code that could be made, but @Iharob has already done a good job of addressing those in his answer.

As I see it, you need to improve the logic of your code, for instance

1. Don't increment PicCounter yet
2. You have a mismatch in your pointers indirection level, in the caller function you have struct Picture * and you are passing it as struct Picture ** to a function taking struct Picture ***. You should reason about it and what is it that you really want. It seems that you want an array of struct Picture *.

Style,

1. Don't use PicCounter for a variable, use number_of_pictures or something more appropriate.
2. Don't write 1 * sizeof(...) it's too explicit and you don't need to be as much.

Tooling,

1. Enable compiler warnings to spot simple mistakes that you make without noticing, or those that are a conceptual flaw like the mismatched indirection of pointers in your code.

That said, I think you probably wanted to write this

int
AddPicture(struct Picture ***tab, int *counter)
{
    struct Picture **array;   
    array = realloc(*tab, (*counter + 1) * sizeof(*array));
    if (array != NULL)
    {
        struct Picture *current;
        // We will avoid using the `tab' variable because it's
        // confusing, the only reason why it's a *** pointer is
        // to do this here, and below where we set it to `NULL'
        *tab = array;
        // We also need to allocate space for a new image to store
        // the results in.
        current = malloc(sizeof(*current));
        if (current == NULL)
            goto error;    
        // This is a guess, but isn't the code just initializing
        // the `pic' that you allocated on the stack?
        //
        // Well, the following is the same.
        //
        // Oh, and should we check for errors here?
        UploadFile(current);            
        // Now we can increase counterand update our array
        array[(*counter)++] = current;
        // We have succeded, so return 0
        return 0;
    }
error:
    printf("Memory reallocation error");
    // Release memory allocated so far
    free(*tab);
    // Prevent dangling pointer
    *counter = 0;
    *tab = NULL;
    // A non zero value means, an error
    return 1;
}

And the caller function should be something like

struct Picture **tabofpics;
int counter = 0;
// `realloc' will take car of the allocation
tabofpics = NULL;
if (AddPicture(&tabofpics, &picounter) == 0)
    // Everything ok, picture added
else
    // Error

Please be careful with *** pointers, they can cause a lot of confusion. I have to think 4 or 5 times about

(*tab)[*counter] = current;

before I can see that it's correct. So I chose to change it as you can see in the code.

Your code should be as descriptive as possible, and using *** pointers goes in the wrong direction of that.