2 Aug
2011
2 Aug
'11
2:24 p.m.
Dear all,
in the if starting at line 1591 in lapdm.c there are two return, maybe you
want to fix this
1591 if (!mctx.dl) {
LOGP(DLAPDM, LOGL_NOTICE, "Received frame for unsupported "
"SAPI %d!\n", sapi);
return -EINVAL;
msgb_free(msg);
return -EIO;
1597 }
Regards,
Loretta
--
View this message in context:
http://baseband-devel.722152.n3.nabble.com/returning-twice-in-lapdm-c-tp321…
Sent from the baseband-devel mailing list archive at Nabble.com.
3 Aug
3 Aug
10:45 a.m.
Hi Loretta,
On Tue, Aug 02, 2011 at 05:24:00AM -0700, screaming-pain wrote:
in the if starting at line 1591 in lapdm.c there are
two return, maybe you
want to fix this
thanks for pointing this out. It's great to know that somebody is
reading the code, after all.
Without checking the git logs, it looks to me like a merge conflict
might have been resolved the wrong way.
But actually, where did you find this? libosmocore.git only has the
"return -EIO" in src/gsm/lapdm.c
Regards,
Harald
--
- Harald Welte <laforge(a)gnumonks.org> http://laforge.gnumonks.org/
============================================================================
"Privacy in residential applications is a desirable marketing option."
(ETSI EN 300 175-7 Ch. A6)
10:54 a.m.
Hi,
It was.
But there is both in the osmocom-bb tree:
src/shared/libosmocore/src/gsm/lapdm.c
src/host/layer23/src/common/lapdm.c
The latter has the problem.
Cheers,
Sylvain
But actually,
where did you find this? libosmocore.git only has the
"return -EIO" in src/gsm/lapdm.c
I think it is an issue found by smatch, maybe the copy in osmocomBB was not
updated yet.
12:05 p.m.
Hi Sylvain and others,
On Wed, Aug 03, 2011 at 10:54:56AM +0200, Sylvain Munaut wrote:
But there is both in the osmocom-bb tree:
src/shared/libosmocore/src/gsm/lapdm.c
src/host/layer23/src/common/lapdm.c
The latter has the problem.
I think if there still is code in osmocom-bb that uses a private copy,
it should be updated to use the code from libosmocore/libosmogsm
instead.
We really don't want to have copies lying around.
--
- Harald Welte <laforge(a)gnumonks.org> http://laforge.gnumonks.org/
============================================================================
"Privacy in residential applications is a desirable marketing option."
(ETSI EN 300 175-7 Ch. A6)
12:09 p.m.
Hi all,
hehe, I read the code every day, I don't always understand it though :)))
Anyway, as Sylvain pointed out that bit of code is in
src/host/layer23/src/common/lapdm.c
Have a good day,
Loretta
--
View this message in context:
http://baseband-devel.722152.n3.nabble.com/returning-twice-in-lapdm-c-tp321…
Sent from the baseband-devel mailing list archive at Nabble.com.
4 Aug
4 Aug
8:34 a.m.
I fear you don't have the time, but we'd definitely appreciate if somebody put
some documentation in the wiki abou the structure of the software. many years ago i wrote
a document about the journey of a packet through the linix kernel network stack. It helped
me a lot to fully understand it myself...
Regards
--
Sent from a mobile device, excuse my short response
10:50 a.m.
Harald Welte-3 wrote:
I fear you don't have the time, but we'd definitely appreciate if somebody
put some documentation in the wiki abou the structure of the software.
many years ago i wrote a document about the journey of a packet through
the linix kernel network stack. It helped me a lot to fully understand it
myself...
It's a nice idea, I would like to do something like that, may be it will not
be exhaustive but I will try to write down something when I will have some
spare time.
Regards,
Loretta
--
View this message in context:
http://baseband-devel.722152.n3.nabble.com/returning-twice-in-lapdm-c-tp321…
Sent from the baseband-devel mailing list archive at Nabble.com.
4660
days inactive
4662
days old
baseband-devel@lists.osmocom.org
7 comments
4 participants
participants (4)
-
Harald Welte -
Holger Hans Peter Freyther -
screaming-pain -
Sylvain Munaut