3:18 p.m.
阿部と申します。始めまして。
21.2.37のsrc/dumper.cをみていて気が付いたのですが、pdump_file_get()中
に出てくる pdump_fd は fd だと思うのですが？
diff -ur xemacs-21.2.37.org/src/dumper.c xemacs-21.2.37/src/dumper.c
--- xemacs-21.2.37.org/src/dumper.c Tue Oct 3 08:27:01 2000
+++ xemacs-21.2.37/src/dumper.c Sun Dec 3 16:52:56 2000
＠＠ -1238,9 +1238,9 ＠＠
pdump_mallocadr = xmalloc(pdump_length+255);
pdump_free = pdump_file_free;
pdump_start = (char *)((255 + (unsigned long)pdump_mallocadr) & ~255);
- read (pdump_fd, pdump_start, pdump_length);
+ read (fd, pdump_start, pdump_length);
- close (pdump_fd);
+ close (fd);
return 1;
}
#endif /* !WIN32_NATIVE */
--
Hiroaki Abe
4:14 a.m.
>>>> "Yoshiki" == Yoshiki Hayashi
<yoshiki(a)xemacs.org> writes:
Yoshiki> Hiroaki Abe <h-abe(a)pc.highway.ne.jp> writes:
> 21.2.37のsrc/dumper.cをみていて気が付いたのですが、
> pdump_file_get()中に出てくる pdump_fd は fd だと思うのですが？
Yoshiki> どうもそのようですね。xemacs-patches に出して頂けないでしょ
Yoshiki> うか。dumper だと Olivier さんが反応してくれると思います。
l.176 を参照してください。
static int pdump_fd;
--
University of Tsukuba Tennodai 1-1-1 Tsukuba 305-8573 JAPAN
Institute of Policy and Planning Sciences Tel/fax: +81 (298) 53-5091
_________________ _________________ _________________ _________________
What are those straight lines for? "XEmacs rules."
5:28 a.m.
>>>> "Yoshiki" == Yoshiki Hayashi
<yoshiki(a)xemacs.org> writes:
Yoshiki> Hiroaki Abe
<h-abe(a)pc.highway.ne.jp> writes:
>> 21.2.37のsrc/dumper.cをみていて気が付いたのですが、
>> pdump_file_get()中に出てくる pdump_fd は fd だと思うのですが？
Yoshiki> どうもそのようですね。xemacs-patches に出して頂けないでしょ
Yoshiki> うか。dumper だと Olivier さんが反応してくれると思います。
>>>> "Stephen" == Stephen J Turnbull
<turnbull(a)sk.tsukuba.ac.jp> writes:
Stephen> l.176 を参照してください。
Stephen> static int pdump_fd;
Oops. 以下の部分は省略してしまったのです。
dumper.c のなかでは pdump_file_get は load time しか使わないのですが
load time には pdump_fd は open されていません。逆に fd は close されな
いので pdump_fd -> fd のパッチは正しいと思います。
提出するときにそういう説明を加えるといいでしょう。
--
University of Tsukuba Tennodai 1-1-1 Tsukuba 305-8573 JAPAN
Institute of Policy and Planning Sciences Tel/fax: +81 (298) 53-5091
_________________ _________________ _________________ _________________
What are those straight lines for? "XEmacs rules."
12:24 p.m.
The following patch is for src/dumper.c.
It seems that pdump_fd isn't used when pdump_file_get() is called, so
read() fails and fd isn't closed.
diff -ur xemacs-21.2.37.org/src/dumper.c xemacs-21.2.37/src/dumper.c
--- xemacs-21.2.37.org/src/dumper.c Tue Oct 3 08:27:01 2000
+++ xemacs-21.2.37/src/dumper.c Sun Dec 3 16:52:56 2000
＠＠ -1238,9 +1238,9 ＠＠
pdump_mallocadr = xmalloc(pdump_length+255);
pdump_free = pdump_file_free;
pdump_start = (char *)((255 + (unsigned long)pdump_mallocadr) & ~255);
- read (pdump_fd, pdump_start, pdump_length);
+ read (fd, pdump_start, pdump_length);
- close (pdump_fd);
+ close (fd);
return 1;
}
#endif /* !WIN32_NATIVE */
--
Hiroaki Abe
6:46 p.m.
On Mon, Dec 04, 2000 at 09:24:05PM +0900, Hiroaki Abe wrote:
The following patch is for src/dumper.c.
It seems that pdump_fd isn't used when pdump_file_get() is called, so
read() fails and fd isn't closed.
Reviewed, approved and committed. Thanks.
OG.
PS: It would be nice if you could add a Changelog next time :-)
8541
days inactive
8542
days old
5 comments
4 participants
participants (4)
-
Hiroaki Abe -
Olivier Galibert -
Stephen J. Turnbull -
Yoshiki Hayashi