[groonga-dev,00285] [PATCH] grn_view_add()のviewにNULLを渡すと落ちる

Back to archive index

Kouhei Sutou kou****@clear*****
2009年 11月 12日 (木) 16:48:08 JST


須藤です。

grn_view_add()のviewのNULLチェックがもれているので、
/d/view_addでviewに存在しないオブジェクトの名前を渡したりす
ると落ちてしまいます。

diff --git a/lib/db.c b/lib/db.c
index 0b9819f..6baaa20 100644
--- a/lib/db.c
+++ b/lib/db.c
@@ -1048,7 +1048,7 @@ grn_view_transcript(grn_ctx *ctx, const char *path,
 grn_id
 grn_view_add(grn_ctx *ctx, grn_obj *view, grn_obj *table)
 {
-  if (view->header.type != GRN_TABLE_VIEW) {
+  if (!view || view->header.type != GRN_TABLE_VIEW) {
     ERR(GRN_INVALID_ARGUMENT, "invalid view");
     return ctx->rc;
   }

-- 
--------------------------------------------------------
各種ブラウザ向けツールバーやプラグインの開発を承ります。
また、ブラウザからICカードやカメラなどのハードウェアへ
アクセスする機能の開発も承ります。
--------------------------------------------------------
  株式会社クリアコード 須藤功平 <kou****@clear*****>
    Tel: 03(6231)7270 URL: http://www.clear-code.com/
--------------------------------------------------------




groonga-dev メーリングリストの案内
Back to archive index