Chronological Thread 

< Chronological >      < Thread >

---

"Joris Robijn"
<joris AT robijn.net>
wrote:
> Rene,
>
> I saw (while debugging) that the use of the configfile functions in
> lcdm001.c was not yet completely safe, the string should be copied
> (it does currently work, but for how long...). I have made the
> comment in the configfile.h file a bit clearer.
>
> char *config_get_string( char * sectionname, char * keyname,
> int skip, char * default_value );
> // Returns a pointer to the string associated with the specified key.
> // The string should never be modified, and used only short-term. You
> can
> // for example scan it or copy it. In successive calls this function
> can
> // re-use the data space !
>
> Can you please copy the string ?

Yup! I've just modified the sources ;)

BTW: Is a strcpy OK?

In another version I've changed the whole way my driver handles data
(private_data struct), so I will have to modify that version, too, in order
to commit it.
I'll probably do so tomorrow ;)

Thanks,

Rene

---

• [Lcdproc] config_get_string, Joris Robijn, 12/02/2001
  • [Lcdproc] config_get_string, Rene Wagner, 12/02/2001
    • [Lcdproc] config_get_string, Joris Robijn, 12/02/2001
      • [Lcdproc] config_get_string, <dglaudemailing AT gmx.net (David GLAUDE Mailing), 12/03/2001
        • [Lcdproc] config_get_string, Rene Wagner, 12/03/2001
          • [Lcdproc] config_get_string, Joris Robijn, 12/04/2001
    • [Lcdproc] config_get_string, <dglaudemailing AT gmx.net (David GLAUDE Mailing), 12/03/2001

---

Archive powered by MHonArc 2.6.18.