NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit7fb8801

committed

Clear errno before calling strtol() in spell.c.

Per POSIX, a caller of strtol() that wishes to check for errors mustset errno to 0 beforehand. Several places in spell.c neglected that,so that they risked delivering a false overflow error in case errnohad been ERANGE already. Given the lack of field reports, this casemay be unreachable at present --- but it's surely trouble waiting tohappen, so fix it.Author: Jacob Brazeal <jacob.brazeal@gmail.com>Discussion:https://postgr.es/m/CA+COZaBhsq6EromFm+knMJfzK6nTpG23zJ+K2=nfUQQXcj_xcQ@mail.gmail.comBackpatch-through: 13

1 parent67fc4c9 commit7fb8801Copy full SHA for 7fb8801

File tree

1 file changed

-0

lines changed

src/backend/tsearch
- spell.c

1 file changed

-0

lines changed

`‎src/backend/tsearch/spell.c`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -375,6 +375,7 @@ getNextFlagFromString(IspellDict Conf, const char sflagset, char sflag)`
`375`	`375`	`stop= (maxstep==0);`
`376`	`376`	`break;`
`377`	`377`	`caseFM_NUM:`
	`378`	`+errno=0;`
`378`	`379`	`s=strtol(*sflagset,&next,10);`
`379`	`380`	`if (*sflagset==next\|\|errno==ERANGE)`
`380`	`381`	`ereport(ERROR,`
`@@ -1037,6 +1038,7 @@ setCompoundAffixFlagValue(IspellDict Conf, CompoundAffixFlag entry,`
`1037`	`1038`	`char*next;`
`1038`	`1039`	`inti;`
`1039`	`1040`
	`1041`	`+errno=0;`
`1040`	`1042`	`i=strtol(s,&next,10);`
`1041`	`1043`	`if (s==next\|\|errno==ERANGE)`
`1042`	`1044`	`ereport(ERROR,`
`@@ -1164,6 +1166,7 @@ getAffixFlagSet(IspellDict Conf, char s)`
`1164`	`1166`	`intcuraffix;`
`1165`	`1167`	`char*end;`
`1166`	`1168`
	`1169`	`+errno=0;`
`1167`	`1170`	`curaffix=strtol(s,&end,10);`
`1168`	`1171`	`if (s==end\|\|errno==ERANGE)`
`1169`	`1172`	`ereport(ERROR,`
`@@ -1740,6 +1743,7 @@ NISortDictionary(IspellDict *Conf)`
`1740`	`1743`
`1741`	`1744`	`if (*Conf->Spell[i]->p.flag!='\0')`
`1742`	`1745`	`{`
	`1746`	`+errno=0;`
`1743`	`1747`	`curaffix=strtol(Conf->Spell[i]->p.flag,&end,10);`
`1744`	`1748`	`if (Conf->Spell[i]->p.flag==end\|\|errno==ERANGE)`
`1745`	`1749`	`ereport(ERROR,`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit7fb8801

File tree

1 file changed

1 file changed

`‎src/backend/tsearch/spell.c`

0 commit comments