Re: possible seg fault

From:

James Kanze <james.kanze@gmail.com>

Newsgroups:

comp.lang.c++

Date:

Mon, 17 Sep 2007 07:24:57 -0000

Message-ID:

<1190013897.013127.77410@19g2000hsx.googlegroups.com>

On Sep 17, 12:52 am, "runner" <emb@dded> wrote:

<jhagen...@fastermail.com> wrote in message
news:1189982423.968704.154860@r29g2000hsg.googlegroups.com...
On Sep 16, 5:38 pm, jhagen...@fastermail.com wrote:

On Sep 16, 5:27 pm, "Laurent D.A.M. MENTEN"

<laurent.men...@teledisnet.be> wrote:

jhagen...@fastermail.com a =E9crit :

The following program compiles successfully on a
Microsoft Visual C++ 6.0 compiler but when I execute
it..it encounters an unknown error..may be a because of
a seg fault..

#include <string.h>
#include <iostream.h>

int main(int argc, char **argv){
           int i;
           int len;
           char *addressLine[8];

           addressLine[1] = "String1";
           addressLine[2] = "String2";

           for(i = 1; i <= 8; i++){
                   len = strlen(addressLine[i]);
                   if (len >= 1){
                           cout << "Address: " << addressLine[i];
                           cout << "\n";
                   }
           }
   return (0);
}

but when I just use the following without the loop..

cout << "Address: " << addressLine[1];

it just works fine...

You did not initialize entries 3..8 of the addressLine array... their
content is pure garbage...- Hide quoted text -

Thanks! for ur reply..I need to use only 2 entries in the array...How
do I iterate and print by handling the error at the same time..I tried
to use

                   if(!addressLine[i]){
                      //Print value..
                   }
but same problem.

I am sorry I meant..
                   if(addressLine[i]){
                      //Print value..
                   }

addressLine[0] = "String1";
addressLine[1] = "String2";
addressLine[2] = NULL;

First, of course, the obvious solution is to use std::vector<
std::string >; std::vector can be adjusted to the required size.
But given what he's got, he really should write:

char const* addressLine[ 8 ] = { "String1", "String2 } ;

Never, never leave a pointer uninitialized.

--
James Kanze (GABI Software) email:james.kanze@gmail.com
Conseils en informatique orient=E9e objet/
Beratung in objektorientierter Datenverarbeitung
9 place S=E9mard, 78210 St.-Cyr-l'=C9cole, France, +33 (0)1 30 23 00 34