out put the value from structure

#include<开发者_Go百科stdio.h>

struct classifier
{
    char src_address[15];
    char dst_address[15];
    int src_port;
    int  dst_port;
};

main()
{
    int i;
    struct classifier clsf[4];
    struct classifier *ptr;

    for(i = 0; i < 2; i++)
    {
        ptr = & clsf[i];
        scanf("%s",ptr[i].src_address);
        scanf("%s",ptr[i].dst_address);
        scanf("%d",&ptr[i].src_port);
        scanf("%d",&ptr[i].dst_port);
        display(ptr[i]);
    }
}

void display(struct classifier ptr)
{
    printf("\n%s", ptr.src_address );
    printf("\n%s", ptr.dst_address );
    printf("\n%d", ptr.src_port);
    printf("\n%d", ptr.dst_port );
}

the o/p that i get is like this

this is i/p

123.123.123.123 
213.234.234.124
3244
1342

o/p
123.123.123.123213.234.234.124
213.234.234.124
3244
1342 

why is that first value repeating. what is wrong in the code

---

Because you write too much data into your strings.

char src_address[15];

is 15 chars long, including the terminating 0 character, thus you must not put more than 14 characters into it. However, 123.123.123.123 is 15 chars.

Since the two fields src_address and dst_address happen to be stored in consecutive memory blocks, the value written into the latter follows immediately - without a separating 0 byte - the former. Thus printf cannot but print both when you intend to print the former.

You should reserve enough space in your char arrays to incorporate the terminating 0 value (and explicitly ensure that it really is put there!). In general, in real life apps you should defend yourself against buffer overruns caused by too long input.