Hi, On Sat, 26 Jul 2008, Miklos Vajna wrote: > On Sat, Jul 26, 2008 at 05:38:55PM +0200, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: > > > - if (!strcmp(main_cmds.names[i]->name, all_strategy[j].name)) > > > + if (!strncmp(ent->name, all_strategy[j].name, ent->len)) > > > > Oops... that is not what I meant. You'd have to check if > > !all_strategy[j].name[ent->len], too... > > Hmm. So let's say ent->name is "ours.exe", ent->len is set to 4. > > Then !strncmp(ent->name, all_strategy[j].name, ent->len) will be true, > and the command will not be added to the exclude list. What I meant is: if you change the strcmp to strncmp because one of the both strings is not supposed to be NUL terminated, you still want to make sure that one is not a strict prefix of the other. Ciao, Dscho -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
