| 00:45.08 | caio1982 | oh, corydon already solved that... nice... |
| 01:28.06 | *** join/#asterisk-bugs ashert (n=hard@ool-4574b37a.dyn.optonline.net) |
| 01:28.14 | *** part/#asterisk-bugs ashert (n=hard@ool-4574b37a.dyn.optonline.net) |
| 02:06.28 | *** join/#asterisk-bugs oej (n=olle@soll4-125.cust.blixtvik.net) |
| 03:14.43 | *** join/#asterisk-bugs russellb (n=russell@asterisk/developer-and-stable-maintainer/drumkilla) |
| 03:14.43 | *** mode/#asterisk-bugs [+o russellb] by ChanServ |
| 03:17.20 | *** join/#asterisk-bugs Corydon76-dig (i=green@pdpc/supporter/bronze/Corydon76-home) |
| 03:17.20 | *** mode/#asterisk-bugs [+o Corydon76-dig] by ChanServ |
| 06:29.27 | *** join/#asterisk-bugs oej (n=olle@soll4-125.cust.blixtvik.net) |
| 08:15.07 | *** join/#asterisk-bugs oej (n=olle@soll4-125.cust.blixtvik.net) |
| 11:27.36 | *** join/#asterisk-bugs caio1982 (i=caio1982@CAcert-br/caio1982) |
| 12:26.29 | *** join/#asterisk-bugs Victor_Yure (n=aaa@200.166.132.131) |
| 14:20.18 | *** join/#asterisk-bugs russellb (n=russell@asterisk/developer-and-stable-maintainer/drumkilla) |
| 14:20.18 | *** mode/#asterisk-bugs [+o russellb] by ChanServ |
| 14:34.22 | *** join/#asterisk-bugs jsmith (n=jsmith@nat/digium/x-4a68187418a5d27c) |
| 14:34.22 | *** mode/#asterisk-bugs [+o jsmith] by ChanServ |
| 14:57.02 | *** join/#asterisk-bugs putnopvut (n=putnopvu@nat/digium/x-03338182a1f0f626) |
| 14:57.02 | *** mode/#asterisk-bugs [+o putnopvut] by ChanServ |
| 15:05.21 | *** join/#asterisk-bugs jsmith-teaching (n=jsmith@nat/digium/x-acf6c12a6aa54de8) |
| 15:05.21 | *** mode/#asterisk-bugs [+o jsmith-teaching] by ChanServ |
| 15:09.08 | *** join/#asterisk-bugs jcmoore (n=jcmoore@unaffiliated/tgrman) |
| 15:10.26 | putnopvut | M11710 |
| 15:10.27 | MuffinMan | [feedback] [Asterisk] Core/RTP 0011710: RTPs not sent to the correct IP reported by davidcsi http://bugs.digium.com/view.php?id=11710 |
| 15:10.43 | file | yesssss |
| 15:10.44 | putnopvut | file: would this actually be an issue if it's another Asterisk box that's sending the 200 OK? |
| 15:10.57 | putnopvut | Or is it still probably just a configuration issue? |
| 15:11.24 | file | I do not believe it is Asterisk on the other side. |
| 15:11.50 | file | but it could be (maybe direct rtp setup) |
| 15:11.58 | file | what is your take on it? |
| 15:12.14 | putnopvut | Well, it looks like Asterisk is doing exactly what it's supposed to. |
| 15:12.32 | putnopvut | If the gateway is telling Asterisk to send RTP to the "wrong" IP, and Asterisk is doing it, then there's no problem. |
| 15:12.42 | putnopvut | It sounds like the gateway needs to be configured correctly. |
| 15:13.05 | *** join/#asterisk-bugs d1mas (n=chatzill@host-99.dataart.net) |
| 15:13.43 | file | I was thinking it might be a SIP proxy in the middle with RTP proxying and it's misconfigured |
| 15:13.55 | putnopvut | Actually, now that I think about it...I need to check RFC 3261 real quick.. |
| 15:14.24 | putnopvut | And by "real quick" that means it may take a bit. |
| 15:16.39 | d1mas | file: from the fact you closed 0011239 can I assume you are back to T38? :) |
| 15:17.30 | file | nope, I'm just working on bugs |
| 15:17.59 | d1mas | pity |
| 15:18.23 | putnopvut | Eh, I can't find what I was looking for in the rfc, and now that I think about it I think what I was thinking about had to do with dialog establishment and had nothing to do with SDPs. |
| 15:22.59 | file | putnopvut: yeah I don't see anything wrong |
| 15:23.40 | file | oej: your thought on 11710? |
| 15:23.58 | putnopvut | I was thinking there was something you were supposed to ignore in a 200 OK if something different had been presented in a 180 or 183 first, but I think it has nothing to do with SDP, and I could just be completely wrong. |
| 15:24.22 | putnopvut | I think it has something to do with dialogue establishment, so I may be thinking of tags here. |
| 15:24.30 | oej | The 200 OK sets the parameters for the call - regardless of the 18x message that came before |
| 15:24.52 | oej | You can very well receive a 183 with one SDP pointing to one IP and then a 200 OK from a different server with a different SDP |
| 15:25.15 | putnopvut | So in other words, Asterisk is doing nothing wrong based on the situation presented in 11710. |
| 15:25.36 | file | can you two add notes? :D he probably wouldn't be happy if I closed it with a "hey we are doing everything fine" note... probably reopen |
| 15:25.43 | putnopvut | No prob. |
| 15:26.40 | oej | Asterisk is doing nothing wrong |
| 15:26.59 | oej | Never, ever. Asterisk is *the* standard and reference implementation! |
| 15:27.00 | oej | Ha! |
| 15:27.06 | putnopvut | :) |
| 15:27.22 | file | :D |
| 15:28.01 | putnopvut | Note added. |
| 15:28.58 | file | thanks you two |
| 16:09.19 | *** join/#asterisk-bugs russellb (n=russell@asterisk/developer-and-stable-maintainer/drumkilla) |
| 16:09.20 | *** mode/#asterisk-bugs [+o russellb] by ChanServ |
| 16:59.01 | putnopvut | M11713 |
| 16:59.02 | MuffinMan | [new] [Asterisk] Core-General 0011713: [info] core show locks reported by slavon http://bugs.digium.com/view.php?id=11713 |
| 16:59.06 | putnopvut | wtf... |
| 16:59.08 | Qwell | -8 karma ftl |
| 17:00.22 | caio1982 | and i thought my karma was bad... |
| 17:03.04 | russellb | make that -10 |
| 17:03.09 | Qwell | pwnt |
| 17:03.43 | file | mad props people on bug work, 344 is nifty |
| 17:09.08 | russellb | file: you're nifty <3 |
| 17:09.26 | file | russellb: why thank you sir! |
| 17:09.39 | russellb | file: i figure i owe you after the random mean comment yesterday :-p |
| 17:10.02 | file | russellb: I forgot about that, but now I remember! |
| 17:10.03 | file | </3 |
| 17:10.32 | russellb | sorry :( |
| 17:10.51 | russellb | file: but come on, you know me well enough to where i can mess with you like that |
| 17:11.24 | file | russellb: I know :D |
| 17:11.35 | Qwell | ^ counter-trolling |
| 17:13.55 | JunK-Y | one chance for him karma isnt decreasing exponentially. |
| 17:14.27 | russellb | IMO, he isn't allowed to file bug reports anymore |
| 17:14.30 | russellb | period. |
| 17:14.39 | Qwell | we need a hall of shame |
| 17:15.02 | caio1982 | the karma code has an option for the last 25 guys with bad karma |
| 17:15.10 | Qwell | oh? |
| 17:15.41 | caio1982 | just enable it :) it's on top of one of those karma files |
| 17:15.55 | russellb | ha, really? |
| 17:16.04 | caio1982 | last time i saw the source yeah |
| 17:16.06 | russellb | open source ftw |
| 17:16.11 | Qwell | it does! |
| 17:16.17 | Qwell | SHOW_BOTTOM_SCORES=1 |
| 17:16.17 | russellb | i'd rather do top 5, though |
| 17:16.25 | russellb | don't want to show a bunch of -1's |
| 17:16.29 | Qwell | karma_halloffame.php |
| 17:16.29 | russellb | they don't deserve it |
| 17:16.44 | caio1982 | my ego would like a top 50 :P |
| 17:17.18 | Qwell | russellb: it would be quite easy to add a bottom_scores #, instead of doing 25 of both |
| 17:17.22 | caio1982 | are you going to enable the bottom score s really? |
| 17:17.33 | Qwell | I could do that in like 5 minutes if we want to enable it :p |
| 17:17.39 | caio1982 | +1 |
| 17:17.46 | caio1982 | not karma, just agreeing :P |
| 17:18.46 | russellb | http://bugs.digium.com/karma_halloffame.php |
| 17:18.59 | Qwell | hax |
| 17:19.20 | Qwell | should give a rank too |
| 17:19.45 | file | it seems to be broken |
| 17:19.45 | caio1982 | -13, omg |
| 17:19.47 | mvanbaak | the links are wrong though |
| 17:19.54 | file | mvanbaak: indeed |
| 17:20.10 | Qwell | yeah, using the id from the #25 spot |
| 17:20.15 | mvanbaak | yup |
| 17:20.15 | russellb | ok, disabled until someone fixes :-p |
| 17:20.38 | russellb | i'm too lazy |
| 17:20.57 | mvanbaak | it's php so you're excused ;) |
| 17:22.16 | Qwell | russellb: committed :P |
| 17:22.48 | Qwell | or not |
| 17:23.19 | russellb | updated |
| 17:23.20 | russellb | and enabled |
| 17:23.32 | russellb | you nubbed it up though |
| 17:23.36 | Qwell | hmm |
| 17:23.38 | russellb | :-p |
| 17:23.56 | Qwell | oh, the query is all silly |
| 17:25.05 | Qwell | better? maybe? |
| 17:25.22 | mvanbaak | no id in the link |
| 17:25.27 | Qwell | ay |
| 17:25.28 | Qwell | yay* |
| 17:25.32 | mvanbaak | yay |
| 17:25.56 | Qwell | my uber php skillz |
| 17:27.47 | Qwell | ouch |
| 17:27.56 | Qwell | http://bugs.digium.com/karma_view_transactions.php?id=2000 |
| 17:27.58 | Qwell | -5 from one bug |
| 17:38.23 | *** join/#asterisk-bugs oej (n=olle@soll4-125.cust.blixtvik.net) |
| 17:41.27 | *** join/#asterisk-bugs oej (n=olle@soll4-125.cust.blixtvik.net) |
| 17:45.46 | russellb | who dealt the damage? |
| 17:48.28 | Qwell | you :D |
| 17:48.42 | Qwell | was in 04 I think |
| 17:52.23 | russellb | nice. |
| 18:35.50 | *** join/#asterisk-bugs jsmith (n=jsmith@nat/digium/x-071f2d11db81188f) |
| 18:35.50 | *** mode/#asterisk-bugs [+o jsmith] by ChanServ |
| 19:27.23 | jsmith | russellb: Is bug 11716 related to what you just fixed? |
| 19:28.17 | russellb | M11716 |
| 19:28.18 | MuffinMan | [acknowledged] [Asterisk] Channels/chan_sip/Transfers 0011716: Attended transfer completes but keeps the zap channel with music on hold reported by matheusrossato http://bugs.digium.com/view.php?id=11716 |
| 19:28.25 | russellb | o.O |
| 19:28.29 | file | ...no? |
| 19:28.32 | russellb | unless it's a transfer to parking, probably not |
| 19:28.42 | russellb | s/probably not/no/ |
| 19:29.59 | *** join/#asterisk-bugs Victor_Yure (n=aaa@200.166.132.131) |
| 21:21.56 | putnopvut | file: |
| 21:21.59 | putnopvut | M11712 |
| 21:22.00 | MuffinMan | [new] [Asterisk] Addons/General 0011712: core show channels becomes unresponsive - precursor to deadlock reported by callguy http://bugs.digium.com/view.php?id=11712 |
| 21:22.07 | putnopvut | I think I may know what's wrong here. |
| 21:22.11 | file | do tell |
| 21:22.17 | putnopvut | Check out handle_request_invite. |
| 21:22.33 | putnopvut | Specifically, the part where if (!ast_strlen_zero(p_replaces)) |
| 21:22.52 | file | uh oh, transfer code |
| 21:22.53 | putnopvut | There is a call to get_sip_pvt_byid_locked |
| 21:23.12 | putnopvut | Which locks p->refer->refer_call and p->refer->refer_call->owner |
| 21:23.26 | putnopvut | The problem is that it looks like unless there is an error, those never get unlocked. |
| 21:23.39 | putnopvut | Thus the deadlock experienced when doing a core show channels. |
| 21:23.45 | putnopvut | It can never acquire that channel's lock. |
| 21:24.25 | putnopvut | I figure you know chan_sip better than I do though, so you may know if those locks eventually are unlocked properly. |
| 21:24.46 | file | I believe they get unlocked in handle_invite_replaces |
| 21:28.12 | putnopvut | Hmm, that may be. |
| 21:35.48 | *** join/#asterisk-bugs fakhir (n=fakhir@unaffiliated/fakhir) |
| 21:42.14 | *** topic/#asterisk-bugs by file -> Asterisk Bugs Discussion -=- http://bugs.digium.com/ -=- http://asterisk.org/developers/bug-guidelines - If you are not discussing a report already posted to the bugtracker, the discussion probably belongs in #asterisk. -=- Overall issue count: 342 -=- Monitor new reports in #asterisk-commits -=- Want to help? Test bugs marked Ready For Testing |
| 22:05.50 | *** join/#asterisk-bugs jsmith (n=jsmith@nat/digium/x-1b36a78c67e5df7b) |
| 22:05.50 | *** mode/#asterisk-bugs [+o jsmith] by ChanServ |
| 22:07.50 | Qwell | M11116 |
| 22:07.51 | MuffinMan | [assigned] [Asterisk] Core-General 0011116: [patch] *BSD mutex lock issue reported by ys http://bugs.digium.com/view.php?id=11116 |
| 22:07.54 | Qwell | russellb: didn't you fix that? |
| 22:08.04 | russellb | Qwell: no |
| 22:08.08 | Qwell | oh |
| 22:08.42 | russellb | fixing it requires rewriting how statically defined locks get initialized for non-linux ... |
| 22:08.50 | russellb | his fix was a hack and didn't work on mac |
| 22:08.56 | russellb | i reverted it for non DEBUG_THREADS |
| 22:09.04 | russellb | but DEBUG_THREADS still doesn't build for mac now because of it |
| 22:09.27 | russellb | but since my primary platform of concern is linux, it's pretty low on my priority list ... |
| 22:19.45 | *** join/#asterisk-bugs Victor_Yure (n=aaa@200.166.132.131) |
| 22:28.32 | *** join/#asterisk-bugs caio1982 (i=caio1982@CAcert-br/caio1982) |
| 22:34.03 | Qwell | AAAANNNNDDDD...340 |
| 22:34.21 | *** topic/#asterisk-bugs by file -> Asterisk Bugs Discussion -=- http://bugs.digium.com/ -=- http://asterisk.org/developers/bug-guidelines - If you are not discussing a report already posted to the bugtracker, the discussion probably belongs in #asterisk. -=- Overall issue count: 340 -=- Monitor new reports in #asterisk-commits -=- Want to help? Test bugs marked Ready For Testing |
| 22:34.42 | caio1982 | that's amazing |
| 22:36.27 | caio1982 | JunK-Y: stupid question, did you try some unix2dos conversion on that configure? |
| 22:36.38 | caio1982 | (your cygwin message on -dev) |
| 23:04.38 | putnopvut | file: I'm not 100% convinced that the necessary locks are released in all cases in handle_invite_replaces |
| 23:05.09 | russellb | putnopvut: yeah, the locking there is quite complex :( |
| 23:05.23 | file | it is entirely possible. |
| 23:05.31 | russellb | the locking in handle_blah is complex |
| 23:06.02 | putnopvut | Well, specifically, I think that all exit points need to unlock p->refer->refer_call and replacecall. |
| 23:06.17 | putnopvut | The problem is that several of the first return points do not unlock replacecall at all. |
| 23:06.44 | russellb | well, keep in mind that the problem is that a channel lock is held |
| 23:06.53 | russellb | not to say that there couldn't be a problem with those, too |
| 23:06.59 | putnopvut | replacecall is a channel. |
| 23:07.02 | russellb | orly |
| 23:07.07 | russellb | ast_channel, not sip_pvt ? |
| 23:07.11 | putnopvut | Right. |
| 23:07.16 | russellb | oic |
| 23:07.34 | putnopvut | And it corresponds to the lock which is being contested in 11712's core show locks output. |
| 23:07.44 | russellb | innnnteresting |
| 23:07.50 | jsmith | Kill the locks! |
| 23:07.52 | Qwell | ouuuuuuuteresting too |
| 23:07.52 | jsmith | Kill the locks! |
| 23:07.52 | russellb | well then. |
| 23:08.02 | putnopvut | Yes, but as usual, I'm extremely hesitant to change locking in chan_sip. |
| 23:08.03 | russellb | putnopvut: sounds like it's worth a patch to try :) |
| 23:08.06 | putnopvut | It...scares me. |
| 23:08.10 | russellb | just let someone test it :-p |
| 23:08.13 | russellb | that's what users are for. |
| 23:08.13 | Qwell | jsmith: we should just kill the users, so they stop finding/reporting these bugs :) |
| 23:08.25 | file | jsmith is a user, and I don't want to kill him |
| 23:08.26 | jsmith | Qwell: Wrong... go read http://www.redhatmagazine.com/2008/01/09/the-choice-of-linux |
| 23:08.27 | putnopvut | Yeah, I'll make a patch and ask for some testing. |
| 23:08.30 | Qwell | stop training people to report bugs :( |
| 23:08.31 | Qwell | :P |
| 23:08.34 | jsmith | file: s/user/luser/ |
| 23:08.37 | file | it is hard to get blood out of the carpet |
| 23:08.44 | Qwell | recarpet |
| 23:08.49 | Qwell | or tile |
| 23:11.12 | Qwell | jsmith: yeah, I guess we need users |
| 23:11.18 | Qwell | at least one or two |
| 23:13.17 | JunK-Y | wow, digium for cover of von magazine :) |
| 23:13.27 | Qwell | JunK-Y: oh? |
| 23:13.35 | JunK-Y | caio1982: nope, i did a svn co using tortoisesvn |
| 23:14.38 | jsmith | JunK-Y: Oh? |
| 23:18.25 | JunK-Y | ya, danny windham |
| 23:18.32 | Qwell | oh |
| 23:24.53 | russellb | innnnnteresting. |
| 23:25.30 | russellb | sooo ... codec_g722 converting to sln16 is broken, but my raw app that does it worked |
| 23:25.50 | Qwell | is it using the same 16<>8 path? |
| 23:25.56 | Qwell | g722 can do 16>8 directly |
| 23:26.00 | Qwell | s/can do/does/ |
| 23:26.03 | russellb | no 16-8 path involved here |
| 23:26.09 | Qwell | oh |
| 23:26.11 | russellb | i'm just testing the code to translate from g722 to sln16 |
| 23:26.21 | Qwell | umm, heh |
| 23:26.29 | russellb | g722 == 16 kHz |
| 23:26.30 | Qwell | maybe it's going g722/16 > slin8 > slin16 |
| 23:26.33 | Qwell | that would be funky |
| 23:26.46 | russellb | heh, it couldn't, i unloaded codec_resample |
| 23:26.49 | russellb | so it would have to be ... |
| 23:26.52 | Qwell | oh |
| 23:27.16 | russellb | g722/16 -> sln8 -> g722/16 -> sln8 |
| 23:27.22 | russellb | g722/16 -> sln8 -> g722/16 -> sln16 |
| 23:27.24 | russellb | that's what i mean |
| 23:27.27 | Qwell | yikes |
| 23:27.33 | russellb | that's the only way to get to sln8 and back to 16 |
| 23:27.38 | Qwell | which would be silly, and ... probably ... wouldn't happen |
| 23:27.42 | russellb | right |
| 23:27.46 | Qwell | probably? hopefully? :D |
| 23:27.55 | russellb | sooo ... something appears to be broken in codec_g722. |
| 23:40.49 | caio1982 | JunK-Y: i did a manual svn co and never got this problem... then... :) |
| 23:54.10 | *** join/#asterisk-bugs fakhir (n=fakhir@unaffiliated/fakhir) |