| 00:27.16 | d3wayne | M10915 |
| 00:27.17 | MuffinMan | [feedback] [Asterisk] Channels/General 0010915: Problems when doing an attended and unattended transfer with Thomson Phones reported by ramonpeek http://bugs.digium.com/view.php?id=10915 |
| 00:27.49 | d3wayne | M10696 |
| 00:27.49 | MuffinMan | [closed] [Asterisk] Channels/chan_sip/Transfers 0010696: Doing an attended transfer, asterisk tries multiple times a native bridge reported by tbelder http://bugs.digium.com/view.php?id=10696 |
| 00:28.32 | d3wayne | M10868 |
| 00:28.33 | MuffinMan | [closed] [Asterisk] Channels/chan_sip/Interoperability 0010868: 1.4.11 Stable - Polycom phones hang up when media is re-invited while resuming from an on-hold state reported by mavince http://bugs.digium.com/view.php?id=10868 |
| 00:29.18 | d3wayne | trick or treat! |
| 00:34.28 | d3wayne | file: do you like 10915 ? |
| 00:34.56 | file | like? I like no bugs! |
| 00:36.59 | d3wayne | Bob to Marge, "Madame, your children are nothing but ill-bred troublemakers" |
| 00:37.32 | d3wayne | Homer: "Lisa too?", Bob: "Especially Lisa! But especially Bart!" |
| 00:44.06 | d3wayne | r84990 |
| 00:44.25 | file | A84990 |
| 00:44.27 | MuffinMan | [84990] Committed by file on 2007-10-08 12:03:07: Don't keep trying to native bridge if either of the channels are involved in a masquerade operation to be done. |
| 00:44.29 | d3wayne | ah, thanks |
| 00:45.58 | blitzrage | welcome |
| 02:21.51 | *** join/#asterisk-bugs dlynes (n=dlynes@d154-20-9-152.bchsia.telus.net) |
| 02:51.18 | *** join/#asterisk-bugs ZX81 (n=matt@202.49.106.158) |
| 06:51.37 | *** join/#asterisk-bugs Juggie (n=Juggie@CPE001601df17fb-CM001a6682d7b6.cpe.net.cable.rogers.com) |
| 06:51.38 | *** mode/#asterisk-bugs [+o Juggie] by ChanServ |
| 07:44.31 | *** join/#asterisk-bugs jmls (n=jmls@62.49.235.130) |
| 10:09.49 | *** join/#asterisk-bugs eurodrew555 (n=eurodrew@83-244-164-130.cust-83.exponential-e.net) |
| 11:04.49 | *** join/#asterisk-bugs caio1982 (i=caio1982@CAcert-br/caio1982) |
| 12:07.28 | *** join/#asterisk-bugs blitzrage (n=Leif@asterisk/documenteur-extraordinaire/blitzrage) |
| 12:07.28 | *** mode/#asterisk-bugs [+o blitzrage] by ChanServ |
| 12:21.29 | *** join/#asterisk-bugs dlynes (n=dlynes@d154-20-9-152.bchsia.telus.net) |
| 12:30.16 | blitzrage | jmls: ping? |
| 12:30.26 | blitzrage | M11084 |
| 12:30.26 | MuffinMan | [new] [Asterisk] Core-General 0011084: asterisk segfault reported by jmls http://bugs.digium.com/view.php?id=11084 |
| 12:30.35 | blitzrage | could we get some more information about this? Are you able to reproduce? |
| 12:44.58 | *** join/#asterisk-bugs oej (n=olle@64.251.114.2) |
| 12:46.39 | *** join/#asterisk-bugs oej_ (n=olle@64.251.114.2) |
| 13:38.53 | *** join/#asterisk-bugs eurodrew555 (n=eurodrew@83-244-164-130.cust-83.exponential-e.net) |
| 13:42.09 | *** join/#asterisk-bugs kpfleming (i=kpflemin@nat/digium/x-1b3e9defbcaade3c) |
| 13:42.09 | *** mode/#asterisk-bugs [+o kpfleming] by ChanServ |
| 14:14.08 | blitzrage | anyone got some bugs that need testing? |
| 14:18.38 | Qwell | all of them :D |
| 14:18.59 | Qwell | blitzrage: there is that VM deleted folder one |
| 14:19.07 | blitzrage | what bug num? |
| 14:19.16 | Qwell | M10740 |
| 14:19.17 | MuffinMan | [assigned] [Asterisk] Applications/app_voicemail 0010740: [patch] Move deleted messages to a Deleted Folder reported by ruffle http://bugs.digium.com/view.php?id=10740 |
| 14:19.41 | blitzrage | gotcha -- will check it out! |
| 14:19.58 | Qwell | that's a feature though, but whatever |
| 14:20.14 | blitzrage | still something to work on :) |
| 14:20.17 | Qwell | indeed |
| 14:22.28 | *** join/#asterisk-bugs jcmoore (n=jcmoore@unaffiliated/tgrman) |
| 14:53.22 | blitzrage | Qwell: note added -- looks good! |
| 14:53.38 | *** join/#asterisk-bugs ivanfm (n=ivanfm@c906b486.virtua.com.br) |
| 15:03.10 | Qwell | sweet |
| 15:03.33 | *** join/#asterisk-bugs putnopvut (i=putnopvu@nat/digium/x-98a1f7b0089ab97d) |
| 15:03.33 | *** mode/#asterisk-bugs [+o putnopvut] by ChanServ |
| 15:03.41 | blitzrage | ya I hate him too |
| 15:03.43 | blitzrage | OH putnopvut !!! |
| 15:03.47 | blitzrage | hi :) |
| 15:03.50 | putnopvut | heh |
| 15:09.00 | Qwell | blitzrage: if backupdeleted goes from 3 to 0, and there are 3 messages saved... do you think it should remove the currently existing deleted messages? |
| 15:09.09 | blitzrage | I don't think so |
| 15:09.10 | Qwell | saved in the deleted folder |
| 15:09.11 | Qwell | ok |
| 15:09.19 | blitzrage | it did as I thought it should |
| 15:14.10 | Qwell | cool |
| 15:19.26 | Corydon76-vcch | If you reduce it from 3 to 1, does it delete 3 messages when you add a message to it? |
| 15:19.46 | Qwell | that's a good question |
| 15:21.26 | blitzrage | oh I didn't try that |
| 15:21.31 | blitzrage | let me try |
| 15:22.03 | Corydon76-vcch | Conversely, if you increase it from 3 to 5, and you add a message, it should delete no messages. |
| 15:22.09 | blitzrage | I'll try both of those |
| 15:22.14 | Qwell | the second works |
| 15:22.20 | Qwell | I know I had tested that one |
| 15:23.27 | *** join/#asterisk-bugs zerohalo (n=zeroHalo@pool-71-255-162-167.bstnma.east.verizon.net) |
| 15:23.33 | blitzrage | there is still 3 messages in Deleted |
| 15:23.44 | blitzrage | it just moves everything up it looks like |
| 15:24.00 | Corydon76-vcch | Is that on the increase or the decrease? |
| 15:24.27 | blitzrage | let me clear this out and try it all from a fresh... because this is from 3 -> 0 -> 1 |
| 15:24.28 | *** join/#asterisk-bugs russellb (n=russell@asterisk/developer-and-stable-maintainer/drumkilla) |
| 15:24.28 | *** mode/#asterisk-bugs [+o russellb] by ChanServ |
| 15:27.18 | *** join/#asterisk-bugs miguel3239 (n=elguerom@ns1.nashuacs.com) |
| 15:27.53 | blitzrage | ok, it just moves everything up |
| 15:28.09 | blitzrage | still goes to the end... let me try leaving a bunch of msg's |
| 15:29.14 | blitzrage | ok -- if there are msg's in there, it'll leave them all, and just move them up when you decrease the number from say, 3 to 1 |
| 15:29.46 | Qwell | I'd bet messages 2 and 3 aren't being moved |
| 15:30.05 | Corydon76-vcch | So the routine only deletes a single message, ever, at most? |
| 15:30.12 | Qwell | it appears that way |
| 15:30.18 | Qwell | msgnum+1 probably |
| 15:30.24 | Corydon76-vcch | I'd say that's a bug |
| 15:31.13 | Corydon76-vcch | If you think you're freeing up a bunch of disk space by lowering the tolerance, then the routine should act accordingly |
| 15:31.32 | blitzrage | Qwell: if you have 3 old msgs', then I change backupdeleted=1, then I leave 3 voicemails, then delete them... it keeps moving them up all the way down the list, so now all 3 voicemail msg's are set with a new date |
| 15:31.56 | Qwell | agreed, but I think if you set it to 0, it should disable it, rather than delete the ones that were previously saved |
| 15:31.59 | Corydon76-vcch | and while we're messing with single digits, it's more likely the people using this feature are going to be using it along the lines of reducing from 500 messages to 300 messages |
| 15:32.21 | blitzrage | Qwell: I would agree with that -- it should leave the existing msg's in there |
| 15:32.31 | Corydon76-vcch | Qwell: or -1 to disable |
| 15:32.36 | blitzrage | and I agree with Corydon76-vcch that it should probably free up the extra messages |
| 15:33.24 | Corydon76-vcch | In fact, I think 0 should be reserved as a way to delete all, and -1 is a nonsense number... which would be how to turn it off... |
| 15:34.38 | Corydon76-vcch | or any negative number, really |
| 15:34.50 | Qwell | well, backupdeleted=no |
| 15:35.00 | Qwell | using negative numbers in the config would be a bit silly |
| 15:35.09 | Corydon76-vcch | fair enough |
| 15:35.33 | Corydon76-vcch | but it's stored as an integer... |
| 15:35.35 | Qwell | and...hmm, this should be working. |
| 15:36.10 | Qwell | x = last_message_index(...) + 1;, then it's looping from 1 through x, renaming files until it finds one that doesn't exist |
| 15:37.53 | blitzrage | putnopvut: hey... looks like I have enough information to maybe reproduce 11095 |
| 15:38.00 | putnopvut | All right! |
| 15:38.04 | blitzrage | it was like pulling teeth, but we got it |
| 15:38.15 | putnopvut | If you do, let me know what you did. |
| 15:38.19 | blitzrage | will do |
| 15:38.23 | blitzrage | I'll try after my conference call |
| 15:38.31 | putnopvut | Awesome. |
| 15:45.09 | russellb | M10923 |
| 15:45.10 | MuffinMan | [new] [Asterisk] Channels/chan_sip/General 0010923: crash in ast_var_name on SIP hangup reported by atis http://bugs.digium.com/view.php?id=10923 |
| 15:59.37 | Qwell | wow, 372 |
| 15:59.55 | blitzrage | ya, doing pretty good |
| 15:59.58 | putnopvut | Let's keep up the pace! |
| 16:00.05 | file | I'm happy with things |
| 16:00.14 | blitzrage | 336 in the Asterisk project |
| 16:00.16 | Qwell | hopefully we'll be able to close like 10 chanspy/mixmon bugs |
| 16:00.19 | blitzrage | would be nice to see that at 299 |
| 16:00.21 | Qwell | that'd be hot |
| 16:00.26 | blitzrage | super hawt |
| 16:18.37 | russellb | blitzrage: i may have a test request for you |
| 16:18.43 | blitzrage | russellb: sure! |
| 16:18.43 | russellb | for another patch that may close multiple bugs |
| 16:18.46 | blitzrage | nice |
| 16:18.49 | russellb | but ... still writing it :) |
| 16:18.52 | blitzrage | np! |
| 16:18.55 | blitzrage | I'll be around |
| 16:19.10 | file | blitzrage: if not I know where you live! |
| 16:19.17 | blitzrage | lies |
| 16:21.44 | Qwell | file: what was that warning message people were getting from those weird keepalives? Was it RTP or RTCP read too short? |
| 16:22.41 | file | RTCP |
| 16:22.45 | Qwell | k, thought so |
| 16:25.08 | russellb | file: ok. so open up ast_do_masquerade() |
| 16:25.19 | file | uh oh |
| 16:25.22 | russellb | yes. |
| 16:25.25 | russellb | search for "readq" |
| 16:25.26 | file | trunk or 1.4? |
| 16:25.31 | russellb | 1.4 |
| 16:25.41 | russellb | you'll see a comment that says "Swap the readq's" |
| 16:25.53 | russellb | that code is wrong, and I have it fixed. |
| 16:26.00 | russellb | it is obvious what it is trying to do ... |
| 16:26.07 | russellb | BUT |
| 16:26.13 | russellb | if you go down a little further ... |
| 16:26.31 | file | yes yes |
| 16:26.34 | russellb | there is another block of readq mangling ... which I can not figure out wtf it is supposed to be doing |
| 16:27.14 | file | I don't... understand... that |
| 16:27.18 | russellb | i don't either |
| 16:27.18 | russellb | at all |
| 16:27.30 | russellb | i mean, the code is just plain broken |
| 16:27.37 | russellb | but what i need is to determine the intention |
| 16:27.54 | Qwell | check back before AST_LIST stuff was added? |
| 16:28.02 | file | Qwell: exactly! |
| 16:28.03 | Qwell | I imagine that was when it got broken |
| 16:28.05 | russellb | Qwell: not a bad idea ... |
| 16:28.23 | Qwell | or like 1.2 |
| 16:28.47 | file | I still don't understand what exactly it is doing |
| 16:29.01 | file | rather, why |
| 16:29.06 | russellb | 1.2 was without the macros |
| 16:29.10 | russellb | but it's still weird, yeah |
| 16:29.20 | russellb | let's get this straight. |
| 16:29.29 | russellb | 1) swap the readq between original and clone |
| 16:29.41 | russellb | 2) count the frames in the readq on clone |
| 16:29.59 | russellb | 3) append the original readq to the clone readq |
| 16:30.24 | russellb | 4) move the clone readq to original ... and so the clone has none ... |
| 16:30.29 | russellb | 5) spam the alertpipe |
| 16:30.31 | russellb | WTF |
| 16:30.47 | blitzrage | putnopvut: w00t indeed |
| 16:30.52 | file | I totally do not understand that |
| 16:31.41 | putnopvut | What does AST_LIST_HEAD_SET_NOLOCK do? |
| 16:31.48 | russellb | sets first and last ... |
| 16:32.00 | russellb | the usage of the list macros in 1.4 is broken, which is what i'm fixing |
| 16:32.05 | file | that code is original Marko code |
| 16:32.13 | russellb | but i can't figure out the intention, heh |
| 16:32.20 | putnopvut | Find Marko then. |
| 16:32.54 | russellb | i guess i'll just maintain the current ... behavior |
| 16:33.01 | blitzrage | but I can't even spell right... so ignore me :) |
| 16:33.01 | russellb | but also leave a comment that says "WTF" |
| 16:33.03 | Qwell | XXX it |
| 16:38.51 | *** join/#asterisk-bugs IPetrov (i=IPetrov@ppp85-140-234-50.pppoe.mtu-net.ru) |
| 16:40.15 | *** join/#asterisk-bugs JackEStorm (n=no@ip68-225-77-136.no.no.cox.net) |
| 16:41.13 | blitzrage | putnopvut: I really don't get why this guy keeps giving me the run around -- why can't he just give me a dialplan without me having to "figure out" what he's doing |
| 16:42.19 | putnopvut | I appreciate it, man. |
| 16:42.28 | file | blitzrage: you rock regardless |
| 16:42.34 | blitzrage | heh, thx... I try |
| 16:42.43 | russellb | file: ok, so here is what I think _should_ be done. |
| 16:43.02 | russellb | you have the new channel (original), and the old channel (clone) ... |
| 16:43.32 | russellb | you need to steal any frames from the old channel, and have them at the beginning of the readq on the new one |
| 16:43.40 | russellb | and for any frames you bring over, you need to poke the alert pipe |
| 16:43.46 | russellb | make sense? |
| 16:44.46 | russellb | yay |
| 16:45.55 | putnopvut | So... why would the new channel be called "original" and the old one be called "clone" ? O.o |
| 16:46.09 | putnopvut | That seems bass-ackwards. |
| 16:47.48 | file | putnopvut: remember when I talked about knowing the origin of the code based on the feel of it? |
| 16:47.55 | putnopvut | Yeah. |
| 16:48.04 | file | this is a good example ^_^ |
| 16:50.02 | russellb | putnopvut: i will never be comfortable with that naming |
| 16:50.10 | russellb | it hurts my head now, and always will |
| 16:50.14 | russellb | ... until someone changes it |
| 16:50.27 | file | yeah, I was pondering documenting it in trunk and changing the names ... |
| 16:50.42 | *** join/#asterisk-bugs fakhir (n=fakhir@unaffiliated/fakhir) |
| 16:51.06 | russellb | i'm still trying to wrap my head around this so i get it right |
| 16:51.11 | russellb | it also swaps the alert pipes ... |
| 16:52.05 | file | I would be interested to know what happens if it's removed |
| 16:52.14 | russellb | the swap? |
| 16:52.31 | file | the second piece of code that mucks with the readq that we were discussing above |
| 16:52.44 | russellb | oh ... well ... nothing, probably |
| 16:52.48 | russellb | just the potential to lose a few frames |
| 16:56.32 | blitzrage | putnopvut: ok, so everything works in the latest 1.4 branch -- going to try in 1.4.13 and see if it doesn't work there |
| 16:57.13 | blitzrage | only 2 things now: you've fixed everything since 1.4.13, or, this guy has local issues on his box |
| 16:57.31 | putnopvut | Yeah...no telling. |
| 16:57.50 | putnopvut | Does your teset involve DND in any way? |
| 16:58.05 | blitzrage | oh ya -- he said DND didn't he |
| 16:58.08 | blitzrage | let me try that again |
| 16:58.32 | putnopvut | Yeah, remember he "takes a phone off DND" |
| 16:58.45 | putnopvut | (of course he never mentioned any phones on DND to begin with) |
| 16:58.45 | blitzrage | yep, I remember now |
| 16:58.48 | blitzrage | right |
| 16:58.52 | blitzrage | again - pulling of teeth |
| 17:11.35 | *** join/#asterisk-bugs dlynes (n=dlynes@d154-20-9-152.bchsia.telus.net) |
| 17:44.56 | *** join/#asterisk-bugs oej (n=olle@64.251.114.2) |
| 17:45.33 | blitzrage | russellb: gonna need me soon to test? Or can I go eat? :) |
| 18:01.58 | russellb | blitzrage: go ahead and eat |
| 18:02.02 | russellb | haven't committed it yet |
| 18:02.06 | blitzrage | coolio |
| 18:02.08 | russellb | on teh phone. |
| 18:02.15 | blitzrage | just about to go get something now |
| 19:10.05 | putnopvut | M10885 |
| 19:10.06 | MuffinMan | [feedback] [Asterisk] Applications/app_queue 0010885: Queue recording via monitor-format and monitor-join produces output different from MixMonitor reported by Alric http://bugs.digium.com/view.php?id=10885 |
| 19:10.32 | putnopvut | I've already taken the action of deprecating the monitor-join option in app_queue due to this issue, but that doesn't actually fix the issue at hand. |
| 19:11.04 | putnopvut | I was wondering how bad an idea it would be if someone uses monitor-join to just instead use MixMonitor? |
| 19:11.19 | file | huh? |
| 19:11.28 | putnopvut | Yeah, my wording sucks. |
| 19:12.18 | putnopvut | And now that I think about it, my idea wasn't so hot anyways. |
| 19:12.35 | putnopvut | So, nevermind. |
| 19:30.17 | russellb | blitzrage: see A88153 ... that's what i was hoping you could test at some point. Just run calls through it. Bonus points if the calls involve transfers, or other things that involve masquerades. Even more bonus points for using IAX2 in the picture. |
| 19:30.19 | russellb | A88153 |
| 19:30.21 | MuffinMan | [88153] Committed by russell on 2007-11-01 15:30:10: The readq handling in ast_do_masquerade() got broken when the code was converted |
| 19:30.21 | MuffinMan | sues #10936, #10595, #10368, and the 2nd backtrace of #11084) |
| 19:30.30 | russellb | o.O |
| 19:38.24 | putnopvut | MuffinMan... |
| 19:43.08 | file | there aren't any good SVN modules for perl |
| 19:45.02 | russellb | heh |
| 19:45.10 | russellb | file: it's ok <3 MuffinMan |
| 19:46.31 | putnopvut | M10585 |
| 19:46.32 | MuffinMan | [new] [Asterisk] Core/ManagerInterface 0010585: [RFC] export attended transfer functionality over AMI reported by ornati http://bugs.digium.com/view.php?id=10585 |
| 19:47.30 | jmls | heh. |
| 19:47.34 | jmls | blitzrage: pong. |
| 19:47.41 | putnopvut | Is there any reason not to merge this? |
| 20:04.03 | blitzrage | jmls: ack pong |
| 20:05.07 | blitzrage | russellb: sweet -- I'll try it out when I get home if it's not too late, if not, first thing tomorrow morning -- I can do all over that except IAX2 stuff |
| 20:05.29 | jmls | blitzrage: pong back |
| 20:05.30 | jmls | M11084 |
| 20:05.31 | MuffinMan | [feedback] [Asterisk] Core-General 0011084: asterisk segfault reported by jmls http://bugs.digium.com/view.php?id=11084 |
| 20:06.21 | blitzrage | jmls: hrmmm... looks like Russell committed something -- you should give it a shot! |
| 20:06.53 | jmls | what version ? |
| 20:19.21 | putnopvut | jmls: that commit was to a team branch of Russell's. There's a link to it in the bugnote. It's based off 1.4 |
| 20:20.03 | russellb | jmls: my fix is definitely related to the 2nd backtrace, that wasn't yours |
| 20:20.08 | russellb | but still potentially related to your crash |
| 20:20.25 | russellb | it's hard to say whether it is related or not ... memory corruption bugs show themselves in weird ways |
| 20:21.23 | putnopvut | blitzrage: are you still around? |
| 20:21.41 | putnopvut | Heh. nope. |
| 21:01.17 | Qwell | M10441 |
| 21:01.18 | MuffinMan | [feedback] [Asterisk] Core/RTP 0010441: RTCP Read Too Short generate strange DTMF tones on call. reported by ygor http://bugs.digium.com/view.php?id=10441 |
| 21:01.23 | Qwell | file: does that look similar to the other thingie? |
| 21:03.09 | file | the keep alive thing? yes |
| 23:53.00 | *** part/#asterisk-bugs zerohalo (n=zeroHalo@pool-71-255-162-167.bstnma.east.verizon.net) |
| 23:58.15 | *** part/#asterisk-bugs jmls (n=jmls@62.49.235.130) |