Is it legal to return a local variable from function.

[hari]

Hi all,
Is it legal to return a local variable from function.

Regards
Hari

[Richard Tobin]

In article <95************ *************** *******@d27g200 0prf.googlegrou ps.com>,
hari <ha********@gma il.comwrote:

>Is it legal to return a local variable from function.

You'll have to be more specific. You can't return variables, only
values. You can certainly return the value of a local variable. You
can't safely return a pointer to a local variable, because it won't
exist after the return.

-- Richard

--
"Considerat ion shall be given to the need for as many as 32 characters
in some alphabets" - X3.4, 1963.

[santosh]

In article
<95************ *************** *******@d27g200 0prf.googlegrou ps.com>,
hari <ha********@gma il.comwrote on Monday 19 Nov 2007 4:57 pm:

Hi all,
Is it legal to return a local variable from function.

Yes. Returning a pointer to it however is dangerous unless the said
local is static.

[arunmib]

On Nov 19, 4:34 pm, santosh <santosh....@gm ail.comwrote:

In article
<952875bd-9cc5-4cac-8ade-6fa37cdb4...@d2 7g2000prf.googl egroups.com>,
hari <haricib...@gma il.comwrote on Monday 19 Nov 2007 4:57 pm:

Hi all,
Is it legal to return a local variable from function.

Yes. Returning a pointer to it however is dangerous unless the said
local is static.

Well, I have a doubt. If I allocate memory for a local pointer and
return the address of the pointer to the calling function. Now will
this be dangerous. For eg:

int main()
{
char *test(int i);
char *tmp = NULL;
int i = 10;
tmp = test(i);
printf("%s\n", tmp);
free(tmp);
return 0;
}

char *test(int i)
{
char *str = (char*)malloc(2 0);
sprintf(str, "i value is: %d", i);
return str;
}

[Joachim Schmitz]

"arunmib" <ar*****@gmail. comschrieb im Newsbeitrag
news:83******** *************** ***********@w28 g2000hsf.google groups.com...

On Nov 19, 4:34 pm, santosh <santosh....@gm ail.comwrote:

>In article
<952875bd-9cc5-4cac-8ade-6fa37cdb4...@d2 7g2000prf.googl egroups.com>,
hari <haricib...@gma il.comwrote on Monday 19 Nov 2007 4:57 pm:

Hi all,
Is it legal to return a local variable from function.

Yes. Returning a pointer to it however is dangerous unless the said
local is static.

Well, I have a doubt. If I allocate memory for a local pointer and
return the address of the pointer to the calling function. Now will
this be dangerous. For eg:

int main()
{
char *test(int i);
char *tmp = NULL;
int i = 10;
tmp = test(i);
printf("%s\n", tmp);
free(tmp);
return 0;
}

char *test(int i)
{
char *str = (char*)malloc(2 0);
sprintf(str, "i value is: %d", i);
return str;
}

In this case it is save to return *str. It is unsafe though to a) cast the
result of the malloc, b) not check whether it succeededs and then writing to
/ read from the unchecked result, c) pass an int to that function that
requires more than 7 characters in decimal

Bye, Jojo

[Richard Heathfield]

arunmib said:

<snip>

Well, I have a doubt. If I allocate memory for a local pointer and
return the address of the pointer to the calling function. Now will
this be dangerous. For eg:

int main()
{
char *test(int i);
char *tmp = NULL;
int i = 10;
tmp = test(i);
printf("%s\n", tmp);
free(tmp);
return 0;
}

char *test(int i)
{
char *str = (char*)malloc(2 0);
sprintf(str, "i value is: %d", i);
return str;
}

First, the bugs. You forgot to #include <stdlib.hand <stdio.h- your
compiler would certainly have warned you that something was amiss, if only
you hadn't silenced it with a useless cast. You assume without
justification that the memory allocation request succeeded.

Okay, now to your question.

You haven't allocated any memory "for a local pointer". You've allocated
some memory, full stop. When you do so, you are given a value, which you
can use for referring to that memory. It's a pointer value (which points
to the beginning of the memory you've allocated). You have stored that
pointer value into a pointer object, which is a very right and proper
thing to do.

You have then returned that value, the value of the pointer object, to the
calling function. You have *not* returned the str object, merely its
value, which - if you will recall - points to the beginning of the memory
you allocated. That's absolutely fine, because that memory you allocated
via malloc (if indeed it succeeded at all) will continue to exist until
you explicitly release it by passing that pointer value to free().

--
Richard Heathfield <http://www.cpax.org.uk >
Email: -http://www. +rjh@
Google users: <http://www.cpax.org.uk/prg/writings/googly.php>
"Usenet is a strange place" - dmr 29 July 1999

[Richard Heathfield]

Joachim Schmitz said:

"arunmib" <ar*****@gmail. comschrieb im Newsbeitrag

<snip>

>>
int main()
{
char *test(int i);
char *tmp = NULL;
int i = 10;
tmp = test(i);
printf("%s\n", tmp);
free(tmp);
return 0;
}

char *test(int i)
{
char *str = (char*)malloc(2 0);
sprintf(str, "i value is: %d", i);
return str;
}

In this case it is save to return *str.

No, it isn't.

<snip>

--
Richard Heathfield <http://www.cpax.org.uk >
Email: -http://www. +rjh@
Google users: <http://www.cpax.org.uk/prg/writings/googly.php>
"Usenet is a strange place" - dmr 29 July 1999

[Joachim Schmitz]

"Richard Heathfield" <rj*@see.sig.in validschrieb im Newsbeitrag
news:pd******** *************** *******@bt.com. ..

Joachim Schmitz said:

>"arunmib" <ar*****@gmail. comschrieb im Newsbeitrag

<snip>

>>>
int main()
{
char *test(int i);
char *tmp = NULL;
int i = 10;
tmp = test(i);
printf("%s\n", tmp);
free(tmp);
return 0;
}

char *test(int i)
{
char *str = (char*)malloc(2 0);
sprintf(str, "i value is: %d", i);
return str;
}

In this case it is save to return *str.

No, it isn't.

???
Of course I meant 'return str'

Bye, Jojo

[santosh]

In article
<83************ *************** *******@w28g200 0hsf.googlegrou ps.com>,
arunmib <ar*****@gmail. comwrote on Monday 19 Nov 2007 5:19 pm:

On Nov 19, 4:34 pm, santosh <santosh....@gm ail.comwrote:

>In article
<952875bd-9cc5-4cac-8ade-6fa37cdb4...@d2 7g2000prf.googl egroups.com>,
hari <haricib...@gma il.comwrote on Monday 19 Nov 2007 4:57 pm:

Hi all,
Is it legal to return a local variable from function.

Yes. Returning a pointer to it however is dangerous unless the said
local is static.

Well, I have a doubt. If I allocate memory for a local pointer and
return the address of the pointer to the calling function. Now will
this be dangerous. For eg:

int main()
{
char *test(int i);

Generally it's better to place declarations outside functions.
Specifically, it is needed (not strictly speaking, but calling a
function without a prototype is bad coding for most purposes), if a
function other than main() wants to invoke test().

char *tmp = NULL;
int i = 10;
tmp = test(i);
printf("%s\n", tmp);
free(tmp);
return 0;
}

char *test(int i)
{
char *str = (char*)malloc(2 0);

The cast is not recommended practise in C. It can prevent the compiler
from issuing a diagnostic if a prototype for malloc() is not in scope,
as is the case here. That in turn can cause subtle problems, since the
compiler incorrectly assumes that malloc() returns an int, while,
actually, it returns a void *.

Include stdio.h and stdlib.h before the code for this program.

sprintf(str, "i value is: %d", i);

If the value held in 'i' requires more than seven characters to
represent in decimal this sprintf() call will overwrite memory. For
such cases use snprintf().

return str;

This is fine since what is being returned is the _value_ in 'str',
not 'str' itself, which ceases to exist after this return. The value
however is copied and assigned to 'tmp' in main().

}

[CBFalconer]

arunmib wrote:

>

.... snip ...

>
Well, I have a doubt. If I allocate memory for a local pointer and
return the address of the pointer to the calling function. Now will
this be dangerous. For eg:

int main() {
char *test(int i);
char *tmp = NULL;
int i = 10;
tmp = test(i);
printf("%s\n", tmp);
free(tmp);
return 0;
}

char *test(int i) {
char *str = (char*)malloc(2 0);
sprintf(str, "i value is: %d", i);
return str;
}

You didn't return the address of the pointer, you returned the
pointer. All is well (except for the foolish cast on the return
value of malloc, and the lack of #includes, and the missing void in
the main parameter list).

--
Chuck F (cbfalconer at maineline dot net)
<http://cbfalconer.home .att.net>
Try the download section.

--
Posted via a free Usenet account from http://www.teranews.com