Pointer Warning..

sollte das nicht + *data heißen? ^^

sollte das nicht + *(my_type *) data heißen?
-> also ein Cast zu einem Pointer auf den Typ Deiner Wahl.
Woher soll der Compiler denn sonst wissen, auf was data zeigt,
schließlich ist das ja generisch (könnte z.B. auch ein struct sein)?
Und void - Zeiger darf man nicht einfach dereferenzieren!

Gruß
  Bimmy and Jimmy

Danke für eure schnelle Antworten, vielleicht was ich noch sagen musste 
ist das payload auch einen void* pointer ist.

Ich möchte die zwei Byte im Kopf der Puffer schreiben, das ist aber 
schwachsinnig wie ich das gemacht habe, ich kann doch einen Pointer 
keine Daten zuweisen aber eine Adresse.Habt ihr eine Idee wie ich das am 
besten machen kann?

In diesem Fall kannst Du deinen generischen Pointer in einen
unsigned long Wert umwandeln. Dieser Typ ist per Definition
hinreichend groß, um einen beliebigen Pointer aufzunehmen.
Danach kannst Du diesen Wert ändern und zuweisen.
Kleines Beispiel:

1

#include <stdio.h>

2

3

int bla (void *f)

4

{

5

void *p;

6

unsigned long q;

7

8

        q = (unsigned long) f;

9

        q += 3;

10

        p = (void *)q;

11

12

        printf ("%p => %p\n", f, p);

13

}

14

15

int main (void)

16

{

17

int a = 23;

18

        bla (&a);

19

        return 0;

20

}

Solche Probleme solltest Du im Allgemeinen aber eleganter lösen.
Gruß
Bimmy and Jimmy

Ich danke euch Leute, manchmal braucht man eine Backpfeife um zu 
denken;-),
für die Leute, die sich dafür interessieren wie ich das gelöst habe:

1

void SetData(void* data, unsigned int DataSize)

2

{

3

 struct pbuf p;

4

 unsigned char* SrcBuffer; 

5

 unsigned char* DestBuffer;

6

7

 DestBuffer = (unsigned char*) p->payload;

8

 SrcBuffer  = (unsigned char*)data; 

9

10

 *(UDPBuffer++) = 0x01;

11

 *(UDPBuffer++) = 0x02;

12

13

 for (i = 0; i < DataSize;i++ ) 

14

  *(UDPBuffer++) = *(DataBuffer++);

15

}

@Bimmy and Jimmy:
Danke dir, ich habe deine Antwort erst gesehen nachdem ich gepostet 
habe.

Es gibt aber noch ein Problem, und zwar es wird immer eine Null dazu 
angehängt, woran kann das denn liegen?

Den DataSize-Wert wird mittels SizeOf() bestimt, z.B:

1

unsigned char HAllO_MSG[] ='Hallo';

2

SetData(HAllO_MSG, sizeof(HAllO_MSG));

Gibt es vielleicht eine andere Möglichkeit, wo diese Terminierungsnull 
nicht mitgezählt wird?

> Das kann auch ein konstanter Ausdruck sein.
Ein Beispiel wäre super.
Danke.

Ich habe mir die erste Variante ausgesucht(strlen), vielen Dank!

Wenn *data sowieso in einen (unsigned char*) gecastet wird, warum 
definierst du die Funktion dann nicht auch gleich so? Also:

1

void SetData(unsigned char *data, unsigned int DataSize);

Ich kann übrigens keinerlei Sinn in deinem obigen Code erkennen. Was 
soll der denn eigentlich machen?