Bugs item #1602093, was opened at 2006-11-24 05:12
Message generated for change (Comment added) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=1602093&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
>Status: Pending
>Resolution: Fixed
Priority: 1
Private: No
Submitted By: ClaudiusMaximus (claudiusmaximus)
Assigned to: Miller Puckette (millerpuckette)
Summary: non-executable files installed with executable permissions
Initial Comment:
With pd-0.40-2 from Miller's site:
./configure --enable-jack --prefix=$HOME/puredata-0-40-2 && make && make install
cd $HOME/puredata-0-40-2
find -type f -exec ls -l {} \; | grep "\-rwx" | grep -v "\.pd_linux" | sed "s-^.*\ --g" | grep -v "/bin/" | sort
...gives a list of files that are installed with executable permissions and that are not executable files.
This list is attached.
----------------------------------------------------------------------
>Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2009-01-12 17:56
Message:
following your steps it seems like this is not an issue any more.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=1602093&group_…
Bugs item #1191610, was opened at 2005-04-28 12:28
Message generated for change (Settings changed) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=1191610&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
>Status: Pending
>Resolution: Fixed
Priority: 7
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Miller Puckette (millerpuckette)
Summary: subpatch rename
Initial Comment:
1 create a subpatch (pd sub).
2 create object (ctl 1) click only once into the canvas
(the object is still blue)
3 rename the subpatch to a unknown object (trtrtrtrtr)
4 pd freezes.
marius.schebella_at_chello.at
----------------------------------------------------------------------
>Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2009-01-12 17:53
Message:
seems to be fixed (at least) in 0.42
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-09-16 10:06
Message:
the problem is that glist_noselect() is called recursively and thus it's
inner while-loop does not return (since glist_deselect() immediately quits
and therefore the terminating condition (which should be set within
glist_deselect()) is never met.
a quick and dirty hack is to comment out the line g_editor.c:174 ("if
(reenter) return;" in glist_deselect())
however, the re-entrance code seems to be there on purpose, so i guess, my
"solution" is NOT recommended.
----------------------------------------------------------------------
Comment By: marius schebella (mariusschebella)
Date: 2008-09-16 05:12
Message:
just had a déjà vu. this bug is still freezing Pd.
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2007-10-16 13:35
Message:
Logged In: YES
user_id=564396
Originator: NO
raised priority since it (still) hangs Pd
----------------------------------------------------------------------
Comment By: ClaudiusMaximus (claudiusmaximus)
Date: 2005-11-09 00:33
Message:
Logged In: YES
user_id=769033
I just ran into this bug (or related bug):
1. create a subpatch [pd test]
2. create a couple of number boxes in the subpatch
3. click into the canvas of the subpatch
4. select the number boxes in the subpatch
5. click into the [pd test] box and rename it to the object
[mtof]
6. on clicking into the canvas, Pd hangs:
Pd version 0.39-1
compiled 05:20:44 Nov 7 2005
(gdb) where
#0 glist_deselect (x=0x828b2a0, y=0x8289178) at g_editor.c:215
#1 0x08086de1 in glist_noselect (x=0x828b2a0) at g_editor.c:222
#2 0x0805961c in canvas_free (x=0x828b2a0) at g_canvas.c:873
#3 0x080a4387 in pd_free (x=0x828b2a0) at m_pd.c:33
#4 0x0805c2a9 in glist_delete (x=0x8249318, y=0x828b2a0) at
g_graph.c:120
#5 0x08062b82 in text_setto (x=0x0, glist=0x8249318,
buf=0x4b <Address 0x4b out of bounds>, bufsize=54) at
g_text.c:1293
#6 0x080818c1 in glist_deselect (x=0x8249318, y=0x828b2a0)
at g_editor.c:207
#7 0x08083229 in canvas_doclick (x=0x8249318, xpos=115,
ypos=82, which=1,
mod=137019608, doit=1) at g_editor.c:222
#8 0x080875db in canvas_mousedown (x=0x8295a88, xpos=0,
ypos=0, which=0,
mod=0) at g_editor.c:1252
#9 0x080a54ed in pd_typedmess (x=0x8249318, s=0x82ac0c8,
argc=136614680,
argv=0x42e60000) at m_class.c:728
#10 0x080a53b6 in pd_typedmess (x=0x82bfc90, s=0x82ac0c8,
argc=4,
argv=0x81266e8) at m_class.c:749
#11 0x080a8894 in binbuf_eval (x=0x8295a88,
target=0x82bfc90, argc=0, argv=0x0)
at m_binbuf.c:574
#12 0x080b0cf0 in socketreceiver_read (x=0x8243228, fd=0) at
s_inter.c:540
#13 0x080b28bf in sys_domicrosleep (microsec=136927880,
pollem=1)
at s_inter.c:180
#14 0x080b1f58 in sys_microsleep (microsec=136927880) at
s_inter.c:202
#15 0x080ad6cf in m_scheduler () at m_sched.c:487
#16 0x080b80db in main (argc=136927880, argv=0x8295a88) at
s_entry.c:27
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=1191610&group_…
Patches item #1930733, was opened at 2008-04-01 03:02
Message generated for change (Comment added) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1930733&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: bugfix
>Status: Pending
>Resolution: Fixed
Priority: 7
Private: No
Submitted By: Thomas Grill (xovo)
Assigned to: Miller Puckette (millerpuckette)
Summary: 0.41-4: prevent buffer overrun in m_class.c
Initial Comment:
the array only has MAXPDARG elements.......
--- m_class.ori.c 2008-04-01 03:00:09.000000000 +0200
+++ m_class.c 2008-04-01 03:00:12.000000000 +0200
@@ -763,7 +763,7 @@
va_start(ap, fmt);
while (1)
{
- if (nargs > MAXPDARG)
+ if (nargs >= MAXPDARG)
{
pd_error(x, "pd_vmess: only %d allowed", MAXPDARG);
break;
----------------------------------------------------------------------
>Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2009-01-12 17:34
Message:
fixed in 0.42
----------------------------------------------------------------------
Comment By: Miller Puckette (millerpuckette)
Date: 2008-05-26 18:46
Message:
Logged In: YES
user_id=313747
Originator: NO
It will work fine to allow MAXARG+1 arguments (6) to vmess; the
MAXARGS restriction in typedmess() is unrelated. I think that's
the best way since apparently some code exists that depends on
MAXARG+1. It's not clear what the maximum should really be; since
this is a convenience routine anyway, I'm thinking it should be
expanded, say to 10 or so.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-05-26 17:57
Message:
Logged In: YES
user_id=27104
Originator: NO
It seems to me that the logic of the function needs to change, but adding
+1 to the array will prevent the overflow, though it's not a very clean way
to do it.
Works for now, I suppose.
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-05-26 09:32
Message:
Logged In: YES
user_id=564396
Originator: NO
as thomas has pointed out, there _is_ a buffer overrun which should be
fixed rather than merely unapplying the patch because it gives weird error
messages.
so for the ease of applying it, here is the patch that fixes the leak by
adding an additional element to the array (i'll apply it to pd-ext 0.40)
Index: m_class.c
===================================================================
--- m_class.c (Revision 9919)
+++ m_class.c (Arbeitskopie)
@@ -772,7 +772,7 @@
void pd_vmess(t_pd *x, t_symbol *sel, char *fmt, ...)
{
va_list ap;
- t_atom arg[MAXPDARG], *at =arg;
+ t_atom arg[MAXPDARG+1], *at =arg;
int nargs = 0;
char *fp = fmt;
----------------------------------------------------------------------
Comment By: Thomas Grill (xovo)
Date: 2008-05-26 09:17
Message:
Logged In: YES
user_id=350252
Originator: YES
Is the arg array too small then and must have MAXPDARGS+1 elements
instead?
In the loop, the "at" pointer subsequently points to the elements of
"arg". If the switch statement is reached with a value of "nargs ==
MAXPDARG", then "arg" points to a position outside the array.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-05-25 23:52
Message:
Logged In: YES
user_id=27104
Originator: NO
This patch causes this message when clicking on abstractions and
subpatches in run mode:
error: pd_vmess: only 5 allowed
... you might be able to track this down from the Find menu.
I think this patch might be invalid since nargs is only used to send to
typedmess() as argc, in which case 5 is the appropriate value.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-04-16 01:42
Message:
Logged In: YES
user_id=27104
Originator: NO
checked into branches/pd-extended/0.40
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1930733&group_…
Patches item #1930681, was opened at 2008-04-01 01:44
Message generated for change (Comment added) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1930681&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: bugfix
>Status: Pending
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Thomas Grill (xovo)
Assigned to: Miller Puckette (millerpuckette)
Summary: 0.41-4 s_path.c fixes
Initial Comment:
Two fixes for buffer overrun (buf[1000]=0 later) and rc file args:
--- s_path.ori.c 2008-04-01 01:37:03.000000000 +0200
+++ s_path.c 2008-04-01 01:38:02.000000000 +0200
@@ -379,7 +379,7 @@
int rcargc;
char* rcargv[NUMARGS];
char* buffer;
- char fname[MAXPDSTRING], buf[1000], *home = getenv("HOME");
+ char fname[MAXPDSTRING], buf[1001], *home = getenv("HOME");
int retval = 1; /* that's what we will return at the end; for now, let's think it'll be an error */
/* initialize rc-arg-array so we can safely clean up at the end */
@@ -423,7 +423,7 @@
fclose(file);
if (sys_verbose)
{
- if (rcargv)
+ if (rcargc)
{
post("startup args from RC file:");
for (i = 1; i < rcargc; i++)
----------------------------------------------------------------------
>Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2009-01-12 17:32
Message:
seems to be fixed (though slightly different) in 0.42
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-04-16 01:59
Message:
Logged In: YES
user_id=27104
Originator: NO
checked into branches/pd-extended/0.40
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1930681&group_…
Patches item #1953853, was opened at 2008-04-29 05:40
Message generated for change (Comment added) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1953853&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: bugfix
>Status: Pending
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Jakob Leben (jleben)
Assigned to: Miller Puckette (millerpuckette)
Summary: fix for empty GOP draws red rect on parent
Initial Comment:
This patch fixes the following bug:
The "graph on parent" red rectangle is drawn on the parent patch when an object is created in a subpatch by sending it a message after the subpatch has been set to GOP but it's window has been closed before any objects had been created in it.
This is a patch against pd-0.41-4.
--- pd2patch-orig/g_graph.c 2008-03-15 01:03:00.000000000 +0100
+++ pd2patch-leben/g_graph.c 2008-04-29 01:53:17.000000000 +0200
@@ -43,7 +43,7 @@
&& pd_checkobject(&y->g_pd))
{
x->gl_goprect = 1;
- canvas_drawredrect(x, 1);
+ if(x->gl_havewindow) canvas_drawredrect(x, 1);
}
if (glist_isvisible(x))
gobj_vis(y, x, 1);
----------------------------------------------------------------------
>Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2009-01-12 17:24
Message:
seems to be accepted (or fixed) in 0.42
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-05-01 14:53
Message:
Logged In: YES
user_id=27104
Originator: NO
added patch to demonstrate the bug, and added the patch to
branches/pd-extended/0.40
File Added: red_rect_on_parent_bug.pd
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1953853&group_…
Bugs item #2501709, was opened at 2009-01-12 16:17
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=2501709&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: externals
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Roman Haefeli (reduzent)
Assigned to: Nobody/Anonymous (nobody)
Summary: [any2string] converts non-ascii chars to negative values
Initial Comment:
belongs to: moocow/pdstring/any2string
non-ascii characters (such as the umlaute 'äöü') are converted to negative values.
instead of i, [any2string] converts to i-256, thus the negative values.
[string2any], OTOH, does seem to work correctly. '246' is converted to 'ö'.
this on pd-vanilla linux with:
pdstring version: 0.06 by Bryan Jurish
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=2501709&group_…
Bugs item #2062558, was opened at 2008-08-20 17:12
Message generated for change (Comment added) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=2062558&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: v0.41.0
>Status: Pending
>Resolution: Fixed
Priority: 7
Private: No
Submitted By: Thomas Grill (xovo)
Assigned to: Miller Puckette (millerpuckette)
Summary: 0.41-4 mac binary doesn't accept arguments
Initial Comment:
The MacOS build version 0.41-4 doesn't accept command line arguments when e.g. run as "/Applications/Pd-0.41-4/Contents/Resources/bin/pd -noprefs - lib my_lib". This worked in 0.40-2 and is very handy for debugging purposes.
Please bring back this useful feature!
----------------------------------------------------------------------
>Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2009-01-12 17:10
Message:
it seems like this has finally been fixed in 0.42.2
----------------------------------------------------------------------
Comment By: Miller Puckette (millerpuckette)
Date: 2008-08-21 20:48
Message:
Logged In: YES
user_id=313747
Originator: NO
Wierd... I get this too. As a temporary workaround, instead of working
inside the
"App", just untar the source, ./configure and make -- then the args work,
I think.
I'll try to figure out why they don't when it's an App...
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-08-21 09:35
Message:
Logged In: YES
user_id=564396
Originator: NO
funnily enough we decided yesterday to file a bug-report about this very
issue, but thomas was faster :-)
it is _really_ annoying when doing development/debugging.
interestingly enough, the only argument i found that actually was accepted
is "-help", which gives you all the arguments that are unfortunately
ignored...
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=2062558&group_…
Bugs item #2093470, was opened at 2008-09-04 18:37
Message generated for change (Comment added) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=2093470&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: v0.41.0
>Status: Pending
>Resolution: Duplicate
Priority: 8
Private: No
Submitted By: ClaudiusMaximus (claudiusmaximus)
Assigned to: Miller Puckette (millerpuckette)
Summary: sys_register_loader not exported => no external loaders
Initial Comment:
$ grep -n sys_register_loader pd-0.41-4/src/*
pd-0.41-4/src/s_loader.c:216:void sys_register_loader(loader_t loader)
$
(similar result from pd in SVN as of yesterday)
To be usable by externals (especially on Windows) it should be declared EXTERN in m_pd.h (along with the definition of loader_t).
See for example:
http://lists.puredata.info/pipermail/pd-dev/2007-12/010382.html
----------------------------------------------------------------------
>Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2009-01-12 16:59
Message:
this has been fixed in Pd-0.42;
btw, isn't this a duplication of
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1840431&group_…
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-09-04 21:12
Message:
Logged In: YES
user_id=27104
Originator: NO
There is a patch (#1840431) for this sitting in the patch tracker:
http://sourceforge.net/tracker/index.php?func=detail&aid=1840431&group_id=5…
This affects all platforms, not just Windows.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=2093470&group_…
Bugs item #2501640, was opened at 2009-01-12 16:56
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=2501640&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
Status: Open
Resolution: None
Priority: 1
Private: No
Submitted By: IOhannes m zmlnig (zmoelnig)
Assigned to: Miller Puckette (millerpuckette)
Summary: makefile.in seems to ignore the --datarootdir setting
Initial Comment:
running "./configure" (or "./config.status") always yields an error:
"config.status: WARNING: makefile.in seems to ignore the --datarootdir setting"
configure then creates a hack for @mandir@ (which would depend on @datarootdir@
the attached patch fixes the warning by (additionally) using @datarootdir@
this is with autoconf 2.61 and afaik any "recent" version of Pd.
it's a rather cosmetic error, but why not fix warnings?
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=2501640&group_…